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

Refine polish section impact, define milestones #233

Merged
merged 3 commits into from
Sep 1, 2021

Conversation

eholk
Copy link
Contributor

@eholk eholk commented Aug 26, 2021

This change elaborates the desired impact from the polish area and also
sets an initial set of milestones to bring about this impact. We will
continue to refine these over time.

This change elaborates the desired impact from the polish area and also
sets an initial set of milestones to bring about this impact. We will
continue to refine these over time.
src/vision/roadmap/polish.md Outdated Show resolved Hide resolved
src/vision/roadmap/polish.md Show resolved Hide resolved
| ↳ Implementation | 💤 | |
| Lint: [Must not suspend] | 💤 | |
| ↳ Landed the [RFC] | 💤 | |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs implementation?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll rephrase as "Implemented the RFC." The RFC is already merged, so I think all we need to do is implement what the RFC says.

src/vision/roadmap/polish.md Show resolved Hide resolved
Copy link
Contributor

@nikomatsakis nikomatsakis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@eholk eholk merged commit fec574f into rust-lang:master Sep 1, 2021
@eholk eholk deleted the polish-roadmap branch September 1, 2021 17:32
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