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

Update log messages and comments in "state" machinery #2552

Merged
merged 11 commits into from
Jul 23, 2024

Conversation

jimklimov
Copy link
Member

Primer for work on #266, extracted from PR #2549 which may need more time to think about.

This PR picks the mostly-cosmetic bits of that change that can be applied today without functional risks (and reduce clutter for comparisons of heavier changes in that PR or its descendants which might go another way about this change).

…ide dstate values [networkupstools#266]

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
…c__ and not hard-coded strings [networkupstools#266]

This is more streamlined and also fixes some mis-attributed messages (evolution? copy-paste?)

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
…unc__ and not hard-coded strings [networkupstools#266]

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
…nc__ and not hard-coded strings [networkupstools#266]

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
… NUMBER to help catch code that did not set the flags [networkupstools#266]

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
…BER [networkupstools#266]

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
@jimklimov jimklimov merged commit 95e2d98 into networkupstools:master Jul 23, 2024
30 checks passed
@jimklimov jimklimov deleted the issue-266-intro branch July 23, 2024 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant