-
Notifications
You must be signed in to change notification settings - Fork 55
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
Main #185
base: master
Are you sure you want to change the base?
Main #185
Conversation
Hi, I already did a first round some week/month ago, mainly on the website part and kept the code part for later. So this help is greatly appreciated, thanks! I'll review soon. |
HI, Just to let you know. |
So I had a closer look. Most of the changes are straight forward, but some are not. The Pacemaker project tries itself to improve its wording using I pushed a commit to your branch to fix various technical wording and one bug related to the I would like to hold a bit more this PR to review it again before merging, with fresh eyes. Thanks |
Hi all,
I switched the naming to primary and secondary in documentations, comments, and log messages to get rid of the abusing terms.
This has to be done for some variables, too. But that will require more work and, obviously, more testing.
Just to let you know, I am using "main" in all my projects instead of the abusing term.
Regards,
Stefanie