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

Fix Issue #94 #106

Merged
merged 5 commits into from
Jun 21, 2023
Merged

Fix Issue #94 #106

merged 5 commits into from
Jun 21, 2023

Conversation

lukerg
Copy link
Contributor

@lukerg lukerg commented Jun 20, 2023

Alters behaviour of sub getcurrent to include use of any alternate SNMP port, if present.

@lukerg lukerg requested a review from oetiker as a code owner June 20, 2023 12:43
src/bin/mrtg Outdated Show resolved Hide resolved
Copy link
Owner

@oetiker oetiker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please also update the CHANGES file

lukerg and others added 3 commits June 21, 2023 19:11
@lukerg
Copy link
Contributor Author

lukerg commented Jun 21, 2023

I was uncertain if I ought to bump the version number for this, given that the last merged commit by fweimer-rh did not do so. I have tried to back date both his work and my own but left the version as is.

@lukerg lukerg requested a review from oetiker June 21, 2023 11:32
src/CHANGES Outdated Show resolved Hide resolved
Co-authored-by: Tobias Oetiker <tobi@oetiker.ch>
@lukerg lukerg requested a review from oetiker June 21, 2023 11:55
@oetiker oetiker merged commit c2f79f4 into oetiker:master Jun 21, 2023
1 check passed
@oetiker
Copy link
Owner

oetiker commented Jun 21, 2023

thanks for your contrib

@lukerg lukerg deleted the uptime branch June 23, 2023 23:16
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

2 participants