Re: [Geany-devel] GP HACKING changes

2011-12-18 Thread Enrico Tröger
On Fri, 16 Dec 2011 16:42:53 +0100, Colomban wrote:


>> P.S. Are commit mails working for Geany-Plugins?  I didn't get one
>> for this commit.
>
>I did receive the commit mail for the GP HACKING change so I guess it
>works :)

Already said on IRC, yes, they work.

Matthew, are you subscribed to the geany-plugins-commits mailing list?


Regards,
Enrico

-- 
Get my GPG key from http://www.uvena.de/pub.asc


pgp7XeGzCvn7H.pgp
Description: PGP signature
___
Geany-devel mailing list
Geany-devel@uvena.de
https://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel


Re: [Geany-devel] GP HACKING changes

2011-12-16 Thread Colomban Wendling
Le 16/12/2011 02:56, Matthew Brush a écrit :
> Hi,
> 
> I made some changes[1] to the HACKING file for Geany-Plugins, mostly
> trivial plus I copied and pasted Colomban's committing information from
> Geany's HACKING file.
> 
> I made one addition here, and wanted to see if it's cool with everyone:
> 
>> +* If you're working on a specific plugin, prefix the summary line
>> +  with the lower case name of the plugin (same as the directory name)
>> +  to make it easier to know which commit affected which plugin without
>> +  having to thoroughly examine the commit diff itself.  No prefix is
>> +  needed if the commit is non-plugin specific or affects only files
>> +  outside of the plugins' directories.
> 
> If it's stupid, I can just remove it, but it seems like it might make it
> a little easier, especially for IRC and email commit messages.
> 
> Let me know.

I think it's good :)  We used to have such a prefix, and even though
it's easy with Git to get the history of a single directory, it makes
obvious what part is affected at first glance; so I think it's good.

> P.S. Are commit mails working for Geany-Plugins?  I didn't get one for
> this commit.

I did receive the commit mail for the GP HACKING change so I guess it
works :)

Cheers,
Colomban

> Cheers,
> Matthew Brush
> 
> [1]
> https://github.com/geany/geany-plugins/commit/5afe76c356e96c847556236cdb169878d4d64b9b

___
Geany-devel mailing list
Geany-devel@uvena.de
https://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel


[Geany-devel] GP HACKING changes

2011-12-15 Thread Matthew Brush

Hi,

I made some changes[1] to the HACKING file for Geany-Plugins, mostly 
trivial plus I copied and pasted Colomban's committing information from 
Geany's HACKING file.


I made one addition here, and wanted to see if it's cool with everyone:


+* If you're working on a specific plugin, prefix the summary line
+  with the lower case name of the plugin (same as the directory name)
+  to make it easier to know which commit affected which plugin without
+  having to thoroughly examine the commit diff itself.  No prefix is
+  needed if the commit is non-plugin specific or affects only files
+  outside of the plugins' directories.


If it's stupid, I can just remove it, but it seems like it might make it 
a little easier, especially for IRC and email commit messages.


Let me know.

P.S. Are commit mails working for Geany-Plugins?  I didn't get one for 
this commit.


Cheers,
Matthew Brush

[1] 
https://github.com/geany/geany-plugins/commit/5afe76c356e96c847556236cdb169878d4d64b9b

___
Geany-devel mailing list
Geany-devel@uvena.de
https://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel