-
Notifications
You must be signed in to change notification settings - Fork 701
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
SC2 Documentation: Fix the page titles #3074
SC2 Documentation: Fix the page titles #3074
Conversation
PR #3031 also proposes these modifications. However, it is true that it is technically outside of its scope XD |
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.
LGTM
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.
Looks solid, would slightly reduce the scope of #3031
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.
LGTM, seconding on reducing scope on other PR
What is this fixing or adding?
Fix documentation title
How was this tested?
Documentation-only
If this makes graphical changes, please attach screenshots.
nothing important, just text changes