Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed option description authpassword -> authpasswd + whitespaces #1676

Merged

Conversation

RincewindsHat
Copy link
Member

@waja
Copy link
Member

waja commented Apr 19, 2021

Can you please separate code style fixes from fixing something else. This is mostly an issue when reviewing code changes and backporting fixes to other branches for example.

Many thanks.

@RincewindsHat
Copy link
Member Author

Can you please separate code style fixes from fixing something else. This is mostly an issue when reviewing code changes and backporting fixes to other branches for example.

Many thanks.

I removed the whitespace changes as requested. Thank you :-)

@waja
Copy link
Member

waja commented Apr 20, 2021

@sni do you think we should merge this into maint-2.3?

@sni
Copy link
Contributor

sni commented Apr 20, 2021

yes, why not, it simpy fixes the help output and does not change the behaviour of the plugin itself

@waja waja modified the milestones: 2.3.2, 2.4 Apr 21, 2021
@waja waja self-assigned this Apr 27, 2021
@RincewindsHat
Copy link
Member Author

@waja: ping :-)

@sni sni merged commit c46447a into monitoring-plugins:master Nov 19, 2021
@RincewindsHat RincewindsHat deleted the fix_check_snmp_authpasswd_option branch November 19, 2021 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants