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

update const_generics_defaults release notes #94294

Closed
wants to merge 2 commits into from
Closed

Conversation

lcnr
Copy link
Contributor

@lcnr lcnr commented Feb 23, 2022

probably seems worth explicitly mentioning this, considering that for myself, that's actually the feature i care more about ^^

@rust-highfive
Copy link
Collaborator

r? @Mark-Simulacrum

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive
Copy link
Collaborator

⚠️ Warning ⚠️

  • Pull requests are usually filed against the master branch for this repo, but this one is against stable. Please double check that you specified the right target!

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 23, 2022
@lcnr
Copy link
Contributor Author

lcnr commented Feb 23, 2022

idk if changing this directly on stable is correct

@Mark-Simulacrum
Copy link
Member

No, we don't typically merge to stable unless we're actually meaningfully changing something (since that takes a full bors build cycle).

If you can refile against master (and "back"port the release notes from stable), that would be good. There's a few issues noted in #94278 that might also be nice to patch up on the way.


Unrelated to the contents of this PR, you probably don't want to be creating branches in rust-lang/rust (e.g., lcnr-patch-1). The GitHub UI unfortunately forces this if you edit on the frontend and have write access, but those branches are visible to everyone and are somewhat noisy as a result.

@lcnr lcnr closed this Feb 23, 2022
@lcnr lcnr deleted the lcnr-patch-1 branch February 23, 2022 15:08
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Feb 25, 2022
update const_generics_defaults release notes

supersedes rust-lang#94294

r? `@Mark-Simulacrum`
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.

None yet

3 participants