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
[namespace.def]/1 Remove redundant statement #2853
Conversation
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.
See comment above.
@jensmaurer Should be good now. |
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 good.
Please squash ("rebase -i") the commits and fix the commit comment (and the title of this issue) to reflect the current reality. |
@jensmaurer Should be good now. |
No, you did some kind of merge. Please rebase and force-push instead: "git rebase -i master" |
@jensmaurer: You can also do that as part of merging: "squash and merge" creates a single, merged commit. |
[namespace.def]/1 Applied suggested changes
Alright, now it should be good. Let me know if there is anything else I need to take care of. |
[namespace.def]/1 states:
This is redundant, as global scope is a namespace. This could confuse readers to think that these are two separate things, when they in-fact are not.
Proposed change is to remove "in the global scope".