Skip to content

ExitCode docs improvements#95389

Closed
coolreader18 wants to merge 2 commits intorust-lang:masterfrom
coolreader18:exitcode-docs-improvements
Closed

ExitCode docs improvements#95389
coolreader18 wants to merge 2 commits intorust-lang:masterfrom
coolreader18:exitcode-docs-improvements

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
Contributor

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