Skip to content

Bump edition to 2024 and crate versions to prereleases - #775

Merged
rozbb merged 1 commit into
mainfrom
bump-edition-and-versions
Jul 7, 2025
Merged

Bump edition to 2024 and crate versions to prereleases#775
rozbb merged 1 commit into
mainfrom
bump-edition-and-versions

Conversation

@tarcieri

@tarcieri tarcieri commented Jul 7, 2025

Copy link
Copy Markdown
Contributor

This represents the first breaking change in a new release series, bumping all crates to the 2024 edition of Rust.

As such, the version numbers of all crates have been incremented to represent a new prerelease series:

  • curve25519-dalek: v5.0.0-pre
  • ed25519-dalek: v3.0.0-pre
  • x25519-dalek: v3.0.0-pre

Note that this commit isn't intended to cut associated crate releases of these on crates.io, but is merely bumping the version numbers to denote there are pending breaking changes.

This commit also includes rustfmt changes which were made as part of the
2024 edition.

@tarcieri
tarcieri requested a review from rozbb July 7, 2025 12:56
@tarcieri

tarcieri commented Jul 7, 2025

Copy link
Copy Markdown
Contributor Author

Huh, looks like a possible 2024 edition incompatibility with zeroize_derive. I've run into similar problems with other custom derive crates I maintain.

Perhaps I'll just get rid of the derive and write the impl manually.

Edit: aah, it's because of -Z minimal-versions and it's pulling in an old zeroize_derive. Let me bump the zeroize requirement and see if that fixes the issue.

This represents the first breaking change in a new release series,
bumping all crates to the 2024 edition of Rust.

As such, the version numbers of all crates have been incremented to
represent a new prerelease series:

- `curve25519-dalek`: v5.0.0-pre
- `ed25519-dalek`: v3.0.0-pre
- `x25519-dalek`: v3.0.0-pre

Note that this commit isn't intended to cut associated crate releases of
these on crates.io, but is merely bumping the version numbers to denote
there are pending breaking changes.

This commit also includes rustfmt changes which were made as part of the
2024 edition.

Also includes clippy fixes.
@tarcieri
tarcieri force-pushed the bump-edition-and-versions branch from 2745a29 to 29794e8 Compare July 7, 2025 14:22

@rozbb rozbb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One Q: is the x25519 version bump just bc of the edition bump? Otherwise there’s nothing breaking right

@tarcieri

tarcieri commented Jul 7, 2025

Copy link
Copy Markdown
Contributor Author

Technically the edition bump itself is not breaking per our SemVer policy, however subsequent changes we plan on making (e.g. #762) will be.

If you'd prefer I can extract the version bumps out into a separate PR.

@rozbb

rozbb commented Jul 7, 2025

Copy link
Copy Markdown
Contributor

I think it's fine, ty

@rozbb
rozbb merged commit 53df025 into main Jul 7, 2025
@tarcieri
tarcieri deleted the bump-edition-and-versions branch July 7, 2025 16:16
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.

2 participants