-
Notifications
You must be signed in to change notification settings - Fork 355
fix(docs): correct typos found during code review #1361
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
base: main
Are you sure you want to change the base?
Conversation
Non-functional changes only: - Fixed minor spelling mistakes in comments - Corrected typos in user-facing strings - No variables, logic, or functional code was modified. Signed-off-by: Marcel Petrick <mail@marcelpetrick.it>
|
The files in the |
Signed-off-by: Marcel Petrick <mail@marcelpetrick.it>
Thank you for pointing this out. You are right and I have reverted those two changes. |
debian/changelog
Outdated
| * OS List: Fixed 'Recommended' tag annotation | ||
| * i18n: Added zh-TW Traditional Chinese translation | ||
| * i18n: Updated Korean, German, Ukranian, translations | ||
| * i18n: Updated Korean, German, Ukrainian, translations |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While I appreciate the review - changing previous changelog entries is not something I generally countenance: it feels a little too much like trying to revise history, even if the history you're revising is careless typos.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tdewey-rpi I have reverted those changes for compliance.
(Personally, I have the view that the "current state" should be correct. Even if this means altering the past.)
Signed-off-by: Marcel Petrick <mail@marcelpetrick.it>
|
The |
Non-functional changes only:
Signed-off-by: Marcel Petrick mail@marcelpetrick.it