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
[temp.param] Remove unused class template from example. #3199
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.
If you agree the first change is bogus, I'd suggest to rephrase the commit comment as
[temp.param] Remove unused class template from example.
and force-push the result. Thank you.
Thanks. While the title of this pull request is now beautiful, you've actually queued up two commits, both of which lack a bit of splendor in their commit messages. Use "git log" to see it. Please do a "git rebase -i master", merge the two commits into one, and clean up the commit message. Then do a "git push -u rogerorr -f template-editorial" to force-push and replace everything that was upstream. (We can do that when we merge, but it's always easier if we have less work. Also, I can't merge without a second reviewer anyway, so we might forget eventually.) |
8e0eb38
to
b3474dd
Compare
I have now rebased and force-pushed to get the right commit message and history. |
Jens, can the "change requested" label now be removed? |
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 now.
No description provided.