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

Add a link to CI documentation in rustc-dev-guide #751

Merged
merged 1 commit into from
Jun 13, 2024

Conversation

Kobzol
Copy link
Contributor

@Kobzol Kobzol commented Jun 13, 2024

I recently rewrote and updated the documentation of rust-lang/rust's CI in the rustc-dev-guide (PR, documentation).

I don't think that it's productive to have the documentation on two places, because it can get out of sync (which is exactly what has happened to the Forge and rustc-dev-guide versions recently). So this PR removes the CI documentation from the Forge and replaces it with a link to rustc-dev-guide. I took all the content from the Forge (that was still relevant) and added it to the dev guide, so there should be no documentation lost.

@Kobzol Kobzol requested a review from jdno June 13, 2024 13:39
@rustbot
Copy link
Collaborator

rustbot commented Jun 13, 2024

r? @kennytm

rustbot has assigned @kennytm.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 13, 2024
@kennytm
Copy link
Member

kennytm commented Jun 13, 2024

r? jdno

@rustbot rustbot assigned jdno and unassigned kennytm Jun 13, 2024
@Kobzol Kobzol merged commit 46e9f0b into rust-lang:master Jun 13, 2024
1 check passed
@Kobzol Kobzol deleted the rustc-ci branch June 13, 2024 20:44
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.

4 participants