Re: [Qgis-developer] new commit label: [NEEDS-DOCS] for QGIS commits will also raise an issue in QGIS-Documentation

[Yves Jacolin]

Le mercredi 17 février 2016, 19:53:54 Richard Duivenvoorde a écrit :
> On 17-02-16 19:42, DelazJ wrote:
> > To have a complete list of issues for documentation (especially for
> > needs_docs tag), does it mean that developers should amend their
> > previous commits message (don't know if it's doable)  so that a grep can
> > retrieve these commits? Or do we consider that they are "lost" improveme
> 
> Well, not sure if that is technically possible: it means changing the
> commit message by adding a label AND writing some prose about the
> addition. But dev's feel free to add the label if possible, or else:
> sent an email to the documentation team describing what you fixed/added
> to excisting features :-)
> 
> Regards,
> 
> R

Harrissou,

Note that this is only true for 2.14 release as I copy pasted the commit 
message into each issue for 2.10 and 2.12. Batch script was up for 2.14.

But if I remembered correctly, Otto did a check for 2.14.

Y.
-- 
Yves Jacolin
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [Qgis-developer] new commit label: [NEEDS-DOCS] for QGIS commits will also raise an issue in QGIS-Documentation

[Richard Duivenvoorde]

On 17-02-16 19:42, DelazJ wrote:
> To have a complete list of issues for documentation (especially for
> needs_docs tag), does it mean that developers should amend their
> previous commits message (don't know if it's doable)  so that a grep can
> retrieve these commits? Or do we consider that they are "lost" improveme

Well, not sure if that is technically possible: it means changing the
commit message by adding a label AND writing some prose about the
addition. But dev's feel free to add the label if possible, or else:
sent an email to the documentation team describing what you fixed/added
to excisting features :-)

Regards,

R
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [Qgis-developer] new commit label: [NEEDS-DOCS] for QGIS commits will also raise an issue in QGIS-Documentation

[DelazJ]

Hi,

2016-02-17 14:16 GMT+01:00 Richard Duivenvoorde :

>
> Hi Devs and Community,
>
> As requested here:
>
>
> https://github.com/qgis/QGIS-Documentation/issues/826#issuecomment-184109531
>
> There is a new version of the QGIS commit webhook (script here [0])
> installed.
>
> Besides firing on [feature] (or [FEATURE], script is now
> case-insensitive) the new version also 'listens' to the label
> [needs-docs], giving the developers a way to instruct doc writers to
> please add some extra documentation after a fix or addition to an
> already excisting functionality.
>

Thanks Richard

To have a complete list of issues for documentation (especially for
needs_docs tag), does it mean that developers should amend their previous
commits message (don't know if it's doable)  so that a grep can retrieve
these commits? Or do we consider that they are "lost" improvements reports?


> Please devs use these labels so doc writers have issues to work on and
> have an overview of things to do. BUT please also take time to add some
> text: either in the commit OR in the docs itself.
>
> A big +1. Not always easy to understand how some features should behave,
without intensive tests. The good side is that it helps bug reports :)

Regards,
Harrissou


> Regards,
>
> Richard
>
> [0]
>
> https://github.com/qgis/QGIS-Sysadmin/blob/master/webhooks/github_feature_tracker.cgi
> ___
> Qgis-developer mailing list
> Qgis-developer@lists.osgeo.org
> List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
> Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [Qgis-developer] new commit label: [NEEDS-DOCS] for QGIS commits will also raise an issue in QGIS-Documentation

[Matthias Kuhn]

On 02/17/2016 02:16 PM, Richard Duivenvoorde wrote:
> Besides firing on [feature] (or [FEATURE], script is now
> case-insensitive) the new version also 'listens' to the label
> [needs-docs], giving the developers a way to instruct doc writers to
> please add some extra documentation after a fix or addition to an
> already excisting functionality.
Much appreciated! Thanks Richard

Matthias

-- 
Matthias Kuhn
OPENGIS.ch - https://www.opengis.ch
Spatial • (Q)GIS • PostGIS • Open Source




signature.asc
Description: OpenPGP digital signature
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [Qgis-developer] new commit label: [NEEDS-DOCS] for QGIS commits will also raise an issue in QGIS-Documentation

[Richard Duivenvoorde]

On 17-02-16 14:19, Mathieu Pellerin wrote:
> Can the tag be placed at the end of the commit string or does it have to
> be at the beginning?

Hi Mathieu,

See

https://github.com/qgis/QGIS-Sysadmin/blob/master/webhooks/github_feature_tracker.cgi#L424

it tests for either [feature] or [needs-docs] somewhere in the commit
message, so go wild and place it wherever you think is best :-)

R,
R




___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [Qgis-developer] new commit label: [NEEDS-DOCS] for QGIS commits will also raise an issue in QGIS-Documentation

[Nyall Dawson]

On 18 February 2016 at 00:16, Richard Duivenvoorde  wrote:
>
> Hi Devs and Community,
>
> As requested here:
>
> https://github.com/qgis/QGIS-Documentation/issues/826#issuecomment-184109531
>
> There is a new version of the QGIS commit webhook (script here [0])
> installed.
>
> Besides firing on [feature] (or [FEATURE], script is now
> case-insensitive) the new version also 'listens' to the label
> [needs-docs], giving the developers a way to instruct doc writers to
> please add some extra documentation after a fix or addition to an
> already excisting functionality.

Thank you!

Nyall

>
> Please devs use these labels so doc writers have issues to work on and
> have an overview of things to do. BUT please also take time to add some
> text: either in the commit OR in the docs itself.
>
> Regards,
>
> Richard
>
> [0]
> https://github.com/qgis/QGIS-Sysadmin/blob/master/webhooks/github_feature_tracker.cgi
> ___
> Qgis-developer mailing list
> Qgis-developer@lists.osgeo.org
> List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
> Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [Qgis-developer] new commit label: [NEEDS-DOCS] for QGIS commits will also raise an issue in QGIS-Documentation

[Mathieu Pellerin]

Can the tag be placed at the end of the commit string or does it have to be
at the beginning?
On 17 Feb 2016 20:16, "Richard Duivenvoorde"  wrote:

>
> Hi Devs and Community,
>
> As requested here:
>
>
> https://github.com/qgis/QGIS-Documentation/issues/826#issuecomment-184109531
>
> There is a new version of the QGIS commit webhook (script here [0])
> installed.
>
> Besides firing on [feature] (or [FEATURE], script is now
> case-insensitive) the new version also 'listens' to the label
> [needs-docs], giving the developers a way to instruct doc writers to
> please add some extra documentation after a fix or addition to an
> already excisting functionality.
>
> Please devs use these labels so doc writers have issues to work on and
> have an overview of things to do. BUT please also take time to add some
> text: either in the commit OR in the docs itself.
>
> Regards,
>
> Richard
>
> [0]
>
> https://github.com/qgis/QGIS-Sysadmin/blob/master/webhooks/github_feature_tracker.cgi
> ___
> Qgis-developer mailing list
> Qgis-developer@lists.osgeo.org
> List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
> Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
List info: http://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: http://lists.osgeo.org/mailman/listinfo/qgis-developer