-
Notifications
You must be signed in to change notification settings - Fork 769
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
[range.join.with.iterator] Add missing 'template' keyword prior to dependent template name 'emplace' #5455
Conversation
@jwakely, this looks reasonable (although it clutters the presentation). |
Note that the |
We still need a rebase + force-push. |
Something odd happened with the rebase; we now have 6k lines changes in containers.tex which shouldn't be in this patch. And we have 10 commits. Again, please squash all the changes you want to have in this pull request (and nothing else) to a single commit and force-push. "git rebase -i main" might help. |
Sorry, I think I messed up. |
Because I always submit PRs on the git web page, I haven't actually used git locally for a long long time, so I need to recall these steps. (I've never squash or rebase PRs because I'm not familiar with it, sorry about that. I mostly use perforce system.) |
"This branch has conflicts that must be resolved" |
Thanks for the pointers. |
@jwakely, could you please have a look? Also, this MUST be squashed when merging. The commit history is ... frightening. |
I'm not sure why github thinks it needs rebasing, it squashes cleanly for me. Something like this:
|
Hmm, except that has spurious unrelated changes. What did you do?! |
…pendent template name 'emplace'
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.
Rebased and force pushed to keep just the commit we want.
I don't know, I just did something terrible that I can't remember. I may have And, again, thanks for the instant redemption. |
What I did was:
|
It probably would have been easier to just do:
i.e. just cherry-pick the one good commit from that messy series of 13 commits. Then force-push that to |
No description provided.