Skip to content

Switch back to MSRV 1.82 & 2021 edition.#307

Merged
waywardmonkeys merged 1 commit intolinebender:mainfrom
waywardmonkeys:reduce-msrv
Mar 20, 2025
Merged

Switch back to MSRV 1.82 & 2021 edition.#307
waywardmonkeys merged 1 commit intolinebender:mainfrom
waywardmonkeys:reduce-msrv

Conversation

@waywardmonkeys
Copy link
Contributor

We keep the Rust Style Edition 2024, so the formatting changes previously done remain.

This reverts parts of commit 7e4aa0b.

This is in response to some feedback from potential community users who were concerned about an aggressive MSRV.

We keep the Rust Style Edition 2024, so the formatting changes
previously done remain.

This reverts parts of commit 7e4aa0b.

This is in response to some feedback from potential community users
who were concerned about an aggressive MSRV.
@waywardmonkeys waywardmonkeys added this pull request to the merge queue Mar 20, 2025
Merged via the queue into linebender:main with commit e4de28f Mar 20, 2025
21 checks passed
@waywardmonkeys waywardmonkeys deleted the reduce-msrv branch March 20, 2025 01:01

/// Returns an iterator over the clusters in logical order.
pub fn clusters(&'a self) -> impl Iterator<Item = Cluster<'a, B>> + Clone {
pub fn clusters(&'a self) -> impl Iterator<Item = Cluster<'a, B>> + 'a + Clone {
Copy link
Member

Choose a reason for hiding this comment

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

Something something this should probably use the captures trick, but it's also underdocumented and everyone gets this wrong, and we'll be moving to 1.85 in the reasonably near future anyway.

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.

4 participants