[Nagiosplug-devel] Some pull requests on Github
me at g0hl1n.net
Thu Mar 21 11:41:58 CET 2013
-----BEGIN PGP SIGNED MESSAGE-----
I don't wanted to trigger a new release.
I just wanted to know if my patches follow your guidlines, so I can continue fixing some bugs for you ;-)
So thanks for your acknowledgement Andreas.
And I'll add some more bug-fixing PullRequests for you.
- -------- Original-Nachricht --------
Von: Andreas Ericsson <ae at op5.se>
Gesendet: Thu Mar 21 11:00:00 MEZ 2013
An: Nagios Plugin Development Mailing List <nagiosplug-devel at lists.sourceforge.net>
Betreff: Re: [Nagiosplug-devel] Some pull requests on Github
On 03/20/2013 10:59 PM, Holger Weiß wrote:
> * Richard Leitner <me at g0hl1n.net> [2013-03-20 18:25]:
>> I've currently forked your git repository on Github (Nickname: g0hl1n).
>> There I started to fix some Bugs from the SourceForge.net Bugtracker.
>> For two of them I created the following Pull-Requests on Github:
>> Due to the fact I got no response until now I'd like to ask if it was
>> the correct way to do these changes?
> I've seen your pull requests, and they look just fine to me. We're all
> a bit busy right now (the main project server has to be moved to a new
> home and stuff like that), but I'll probably be able to look into the
> open pull requests (and then preparing a new release) relatively soon.
A new release probably won't be necessary. One patch is a spelling fix
only (which is trivially correct). Adding --timeout to check_nagios is
obviously good, but since it runs locally I doubt it'll ever have time
ACK on both patches from me though, for what it's worth. They're both
correct and follow the plugin coding standards as far as I could see.
Andreas Ericsson andreas.ericsson at op5.se
OP5 AB www.op5.se
Tel: +46 8-230225 Fax: +46 8-230231
Considering the successes of the wars on alcohol, poverty, drugs and
terror, I think we should give some serious thought to declaring war
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
Nagios Plugin Development Mailing List Nagiosplug-devel at lists.sourceforge.net
Unsubscribe at https://lists.sourceforge.net/lists/listinfo/nagiosplug-devel
::: Please include plugins version (-v) and OS when reporting any issue.
::: Messages without supporting info will risk being sent to /dev/null
-----BEGIN PGP SIGNATURE-----
Version: APG v1.0.8
-----END PGP SIGNATURE-----
More information about the Devel