Skip to content
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

vxWorks is using "invalid argument" instead of "Invalid argument" in reporting invalid options #65185

Closed
wants to merge 3 commits into from

Conversation

BaoshanPang
Copy link
Contributor

No description provided.

@rust-highfive
Copy link
Collaborator

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