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.

Sign up for GitHub

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

Jump to bottom

chore: sort Cargo.toml dependencies #415

Merged
merged 1 commit into from
Nov 13, 2023
Merged

chore: sort Cargo.toml dependencies #415

merged 1 commit into from
Nov 13, 2023

Conversation

LNSD
Copy link
Contributor

@LNSD LNSD commented Nov 13, 2023 •
edited
Loading

This is a gardening 🪴 PR sorting all Cargo.toml files' dependencies to avoid merge conflicts.

Note

The dependency section of the cargo.toml files should be changed in an automated way, whenever possible, using the cargo add command since the format may cause issues with the dependency management bots.

@LNSD LNSD requested a review from Theodus November 13, 2023 11:54
@LNSD LNSD self-assigned this Nov 13, 2023
Theodus
Theodus approved these changes Nov 13, 2023
View reviewed changes
Copy link
Member

@Theodus Theodus left a comment

Choose a reason for hiding this comment

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

Thanks!

@Theodus
Copy link
Member

Theodus commented Nov 13, 2023

Merging to include in the release I'm putting out today. CI integration tests are running out of memory on compilation, which I have a workaround for in the release branch

@Theodus Theodus merged commit bb23fe9 into main Nov 13, 2023
1 of 2 checks passed
@Theodus Theodus deleted the chore-sort-cargo-toml-deps branch November 13, 2023 13:53
Theodus pushed a commit that referenced this pull request Dec 18, 2023
@LNSD
chore: sort Cargo.toml dependencies (#415)
70dfc17 
This is a gardening 🪴 PR sorting all `Cargo.toml` files' dependencies to
avoid merge conflicts.

> **Note**
> 
> The dependency section of the cargo.toml files should be changed in an
automated way, whenever possible, using the `cargo add` command since
the format may cause issues with the dependency management bots.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Theodus Theodus Theodus approved these changes

Assignees

@LNSD LNSD

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@LNSD @Theodus