-
Notifications
You must be signed in to change notification settings - Fork 27
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 changelog appendix #511
Conversation
bors try |
tryBuild succeeded: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pushed some commits to reword and reformat the changelog entries a bit. I also configured the lint to prevent having paragraph IDs in this page, which just add noise for no benefit.
Left comments on things that I feel we need to expand on.
|
||
* `Rework \`impl Trait\` lifetime inference <https://github.com/rust-lang/rust/pull/116891/>`_ | ||
|
||
* No change: capturing of lifestime within ``impl Trait`` types is not described in the FLS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This feels like a gap in the FLS we should fill (and describe).
bors try |
tryBuild failed: |
bors try |
tryBuild succeeded: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Marking Lukas's changes as approved, waiting for Lukas to approve my changes before we merge this.
I think the best thing is to merge this PR right now, and tackle the three points I mentioned in #511 (review) in followup PRs that both fix the points and update the changelog.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 (cant approve my own PR)
bors merge |
Build succeeded: |
513: Specify auto trait trait object upcasting r=pietroalbini a=Veykril #511 (comment) Co-authored-by: Lukas Wirth <[email protected]>
512: Add library crate definition back r=pietroalbini a=Veykril #511 (comment) Co-authored-by: Lukas Wirth <[email protected]> Co-authored-by: Pietro Albini <[email protected]>
No description provided.