cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From julio cesar sanchez <>
Subject Re: [proposal] Remove tool generated docs from camera plugin?
Date Sun, 15 Oct 2017 07:52:14 GMT
The already have a section
about how the is generated. But maybe it should be more clear and
say that the shouldn't be updated manually.

BTW, I sent a PR <>
to make the template be in sync with the changes. If anybody can
take a look it would be good. Not sure if I can merge while the tests are
failing, but I only changed the template, so I'm pretty sure I didn't break
anything and tests are failing in all other plugins, but that's another

About if it's a good idea, I've found the original cordova-discuss issue
<> and proposal
<> and reading them I
think it's a good idea as I understand that the API part of the docs are
automatically generated from the js files, which avoids them to be out of
sync if a js change is made.

2017-10-15 1:07 GMT+02:00 Jan Piotrowski <>:

> The "not aware" problem can probably be resolved (80/20) by adding
> comments to the template that are also in the generated output.
> But it the actually a good idea? Is it worth the extra effort?
> -J
> 2017-10-15 0:01 GMT+02:00 julio cesar sanchez <>:
> > cordova-plugin-camera docs work on a different way than other plugins.
> The
> > is generated from jsdoc2md/ when the commit is done.
> > I think it was a proof of concept to be done in all the plugins, but it
> the
> > end it was only done in camera plugin.
> > The problem is most people is not aware of this, so they just update the
> > manually as on any other plugins, and most of the committers
> > aren't aware neither or forgot about it and just merge those changes.
> >
> > So, I was about to send a new PR and the jsdoc2md/ is out of
> > sync with the
> >
> > So, we have two options, we can move back to a regular like the
> > other plugins, or do this in all the plugins and not accept
> file
> > changes without the corresponding jsdoc2md/
> >
> > What do you think?
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message