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

ExitCode docs improvements #95389

Closed

Conversation

coolreader18
Copy link
Contributor

Not sure how changes made during a fcp should be PR'd, obviously this isn't needed for the stabilization so I just based it off the stabilization branch.

r? yaahc

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Mar 27, 2022
@rust-highfive
Copy link
Collaborator

r? @yaahc

(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 Mar 27, 2022
@yaahc
Copy link
Member

yaahc commented Mar 28, 2022

Can you change the base branch from master to my branch in the stabilization PR, yaahc:termination-stabilization-celebration-station?

@coolreader18
Copy link
Contributor Author

Eeek, I don't think so, I'd have to make a new PR in your fork I think.

@coolreader18
Copy link
Contributor Author

I could squash and you could cherry-pick?

@coolreader18 coolreader18 force-pushed the exitcode-docs-improvements branch from 247bbe8 to 4a35b7a Compare March 28, 2022 02:39
@yaahc
Copy link
Member

yaahc commented Mar 28, 2022

I could squash and you could cherry-pick?

Yea, sounds good, thank you!

@yaahc
Copy link
Member

yaahc commented Mar 28, 2022

I've merged the touch up commit, thank you again!

@yaahc yaahc closed this Mar 28, 2022
@coolreader18 coolreader18 deleted the exitcode-docs-improvements branch March 28, 2022 17:28
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. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants