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

Updated instructions for tracking issue #488

Merged
merged 2 commits into from
Nov 1, 2019
Merged

Conversation

alexreg
Copy link
Contributor

@alexreg alexreg commented Oct 31, 2019

Removed link to Forge instructions for documentation since they no longer seem to exist.

Removed link to Forge instructions for documentation since they no longer seem to exist.
@JohnTitor
Copy link
Member

I think it'd be nice to add links to rustc-guide instead like this rust-lang/rust#65262 (comment)

@alexreg
Copy link
Contributor Author

alexreg commented Oct 31, 2019

There's no section in the Rustc guide about writing documentation, last time I checked.

@JohnTitor
Copy link
Member

Uhmm? IMO, it's enough to add the link to here as above.

@alexreg
Copy link
Contributor Author

alexreg commented Oct 31, 2019

Oh, that's useful yes, but it should probably be linked in the previous bullet point (Stabilize the feature), not the documentation one.

@JohnTitor
Copy link
Member

This is ideal for me, what about?

- [ ] Implement the RFC (cc @rust-lang/compiler -- can anyone write up mentoring
instructions?)
- [ ] Adjust documentation ([see instructions on rustc-guide](https://rust-lang.github.io/rustc-guide/stabilization_guide.html#documentation-prs))
- [ ] Stabilization PR ([see instructions on rustc-guide](https://rust-lang.github.io/rustc-guide/stabilization_guide.html#stabilization-pr))

@alexreg
Copy link
Contributor Author

alexreg commented Nov 1, 2019

@JohnTitor Yep, I've updated to that with just minor wording/punctuation differences. Should be good to merge now.

@spastorino spastorino merged commit a448c07 into master Nov 1, 2019
@JohnTitor JohnTitor deleted the alexreg-patch-1 branch November 1, 2019 17:37
@alexreg
Copy link
Contributor Author

alexreg commented Nov 1, 2019

Cheers.

mark-i-m pushed a commit to spastorino/rustc-dev-guide that referenced this pull request Jan 2, 2020
* Updated instructions for tracking issue

Removed link to Forge instructions for documentation since they no longer seem to exist.

* Improved links and wording
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants