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

Ban async-std and old hyper and tokio versions #7380

Merged
merged 3 commits into from
Dec 19, 2024
Merged

Conversation

faern
Copy link
Member

@faern faern commented Dec 19, 2024

In order to somewhat keep our dependency tree, compile times and bugs down, we should avoid certain crates. We already do this, but the list was very short. Here are some more crates that it would be good if we avoided (accidentally?) depending on. As long as there are viable alternatives we should never use these crates.

If we come to a situation where we really have no option, entries here can be discussed of course.


This change is Reviewable

Copy link
Contributor

@MarkusPettersson98 MarkusPettersson98 left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@faern faern force-pushed the ban-more-unwanted-crates branch 2 times, most recently from 121cec8 to 369b038 Compare December 19, 2024 13:43
Copy link
Member

@raksooo raksooo left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@faern faern force-pushed the ban-more-unwanted-crates branch from 369b038 to 2a8442f Compare December 19, 2024 14:30
@faern faern merged commit e73ab2e into main Dec 19, 2024
29 checks passed
@faern faern deleted the ban-more-unwanted-crates branch December 19, 2024 15:06
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.

3 participants