Skip to content

Conversation

@BaoshanPang
Copy link
Contributor

No description provided.

@rust-highfive
Copy link
Contributor

r? @rkruppe

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 7, 2019
@BaoshanPang
Copy link
Contributor Author

cc @n-salim
cc @maya-cm

@hanna-kruppe
Copy link
Contributor

The diff seems fine to rubber-stamp, but Rust has a no merge policy so please rebase ae2bb7b to get rid of the merge commits (preserving that it was authored by @bpangWR, of course).

@BaoshanPang
Copy link
Contributor Author

I have been always using my personal account 'BaoshanPang' to make changes in github, but not sure why sometimes it recognize me as bpangWR which is the company account created by me.

@hanna-kruppe
Copy link
Contributor

Ah, OK, then authorship is no concern, please just get rid of the merge commits in whatever way is most convenient.

@BaoshanPang
Copy link
Contributor Author

Using a new branch is easier for me, I can't find a way to remove the "merge commits" in the current branch.
A new PR is created #65187.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants