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 RELEASES.md for 1.25.0 #48374

Merged
merged 5 commits into from
Mar 22, 2018
Merged

Conversation

XAMPPRocky
Copy link
Member

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 20, 2018
RELEASES.md Outdated
- [`UnsafeCell::into_inner` is now safe.][47204]
- [Implement libstd for CloudABI.][47268]
- [`Float::{from_bits, to_bits}` is now available in libcore.][46931]
- [renamed `ptr::Shared` to `ptr::NonNull`.][46952]
Copy link
Member

Choose a reason for hiding this comment

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

this isn't stable yet, so probably shouldn't be in here, right?

Copy link
Member Author

Choose a reason for hiding this comment

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

@steveklabnik Looking at the beta docs and the prs this mentioned in. It's stable this release unless there was a revert pr I didn't see.

Copy link
Member

Choose a reason for hiding this comment

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

Maybe don't mention ptr::Shared?

Copy link
Member

Choose a reason for hiding this comment

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

yeah, I would make this link to the stabilization of NonNull in that case.

RELEASES.md Outdated
- The borrow checker was sometimes incorrectly permitting overlapping borrows
around indexing operations (see #47349). This has been fixed (which also
enabled some correct code that used to cause errors (e.g. #33903 and #46095).
- [Removed deprecated unstable attribute `#[simd]`.][47251]
Copy link
Member

Choose a reason for hiding this comment

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

same here

Copy link
Member Author

Choose a reason for hiding this comment

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

@steveklabnik @Mark-Simulacrum gave it the relnotes tag so I included it.

RELEASES.md Outdated
- [`Float::{from_bits, to_bits}` is now available in libcore.][46931]
- [renamed `ptr::Shared` to `ptr::NonNull`.][46952]
- [Implement `AsRef<Path>` for Component][46985]
- [Deprecated `[T]::rotate` in favor of `[T]::rotate_{left,right}`.][46777]
Copy link
Member

Choose a reason for hiding this comment

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

rotate_left and rotate_right are not stable yet.

RELEASES.md Outdated
- [`rustdoc` has switched to pulldown as the default markdown renderer.][47398]
- The borrow checker was sometimes incorrectly permitting overlapping borrows
around indexing operations (see #47349). This has been fixed (which also
enabled some correct code that used to cause errors (e.g. #33903 and #46095).
Copy link
Member

Choose a reason for hiding this comment

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

Please add links to these 3 issues.

RELEASES.md Outdated
- [`UnsafeCell::into_inner` is now safe.][47204]
- [Implement libstd for CloudABI.][47268]
- [`Float::{from_bits, to_bits}` is now available in libcore.][46931]
- [renamed `ptr::Shared` to `ptr::NonNull`.][46952]
Copy link
Member

Choose a reason for hiding this comment

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

Maybe don't mention ptr::Shared?

@nikomatsakis
Copy link
Contributor

r? @Mark-Simulacrum

RELEASES.md Outdated
--------
- [Stabilised `#[repr(align(x))]`.][47006] [RFC 1358]
- [You can now use nested groups of imports.][47948]
eg. `use std::{fs::File, io::Read, path::{Path, PathBuf}};`
Copy link
Member

Choose a reason for hiding this comment

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

We should probably use e.g. instead of eg. throughout.

@kennytm kennytm added the T-release Relevant to the release subteam, which will review and decide on the PR/issue. label Feb 21, 2018
[cargo/5029]: https://github.com/rust-lang/cargo/pull/5029
[RFC 1358]: https://github.com/rust-lang/rfcs/pull/1358
[`Location::column`]: https://doc.rust-lang.org/std/panic/struct.Location.html#method.column
[`ptr::NonNull`]: https://doc.rust-lang.org/std/ptr/struct.NonNull.html
Copy link
Contributor

Choose a reason for hiding this comment

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

link returns 404?

Copy link
Member

Choose a reason for hiding this comment

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

The link should be available once 1.25 is released to stable channel.

RELEASES.md Outdated
-----
- [`cargo new` no longer removes `rust` or `rs` prefixs/suffixs.][cargo/5013]
- [`cargo new` now defaults to creating a binary crate, instead of a
libary crate.][cargo/5029]
Copy link
Contributor

Choose a reason for hiding this comment

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

libary -> library

@shepmaster
Copy link
Member

Ping from triage, @Mark-Simulacrum — looks like it's back in your court!

RELEASES.md Outdated
- [`UnsafeCell::into_inner` is now safe.][47204]
- [Implement libstd for CloudABI.][47268]
- [`Float::{from_bits, to_bits}` is now available in libcore.][46931]

Copy link
Member

Choose a reason for hiding this comment

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

Extra line?

RELEASES.md Outdated

Misc
----

Copy link
Member

Choose a reason for hiding this comment

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

Should probably remove this category if it's unused.

@Havvy
Copy link
Contributor

Havvy commented Mar 11, 2018

Isn't this going to be the first release with Rust By Example in the distribution documentation? Shouldn't that be in the release notes?

@XAMPPRocky
Copy link
Member Author

@Havvy Do you have the PR that does that so I can check if it's in beta?

@steveklabnik
Copy link
Member

#46196

but easier: https://doc.rust-lang.org/beta/rust-by-example/

The PR was marked relnotes...

@shepmaster
Copy link
Member

Ping from triage, @Mark-Simulacrum — looks like it's back in your court!

@Mark-Simulacrum
Copy link
Member

Looks like my two nits still stand (#48374 (review)). Otherwise I think we are good to merge.

@XAMPPRocky
Copy link
Member Author

@Mark-Simulacrum I've removed the extra line, and Misc is no longer empty so it shouldn't be removed.

@Mark-Simulacrum
Copy link
Member

@bors r+ p=1

@bors
Copy link
Contributor

bors commented Mar 21, 2018

📌 Commit e2cf172 has been approved by Mark-Simulacrum

@bors
Copy link
Contributor

bors commented Mar 21, 2018

🌲 The tree is currently closed for pull requests below priority 30, this pull request will be tested once the tree is reopened

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 21, 2018
kennytm added a commit to kennytm/rust that referenced this pull request Mar 21, 2018
@pietroalbini
Copy link
Member

@bors rollup

kennytm added a commit to kennytm/rust that referenced this pull request Mar 22, 2018
bors added a commit that referenced this pull request Mar 22, 2018
@alexcrichton alexcrichton merged commit e2cf172 into rust-lang:master Mar 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-release Relevant to the release subteam, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.