Skip to content

chacha20: use cargo hack in CI - #458

Merged
tarcieri merged 2 commits into
masterfrom
chacha20/use-cargo-hack-in-ci
Sep 8, 2025
Merged

tarcieri merged 2 commits into
masterfrom
chacha20/use-cargo-hack-in-ci

Conversation

@tarcieri

@tarcieri tarcieri commented Sep 8, 2025

Copy link
Copy Markdown
Member

To prevent bugs like #454 in the future, exhastively check all feature combinations to ensure they all work correctly

@tarcieri
tarcieri force-pushed the chacha20/use-cargo-hack-in-ci branch from ac639dd to f81d2da Compare September 8, 2025 20:36
@newpavlov

Copy link
Copy Markdown
Member

Maybe it's worth to remove legacy and xchacha features? If you want to discourage their use, we could move the types into a separate module similarly to the hazmat modules.

To prevent bugs like #454 in the future, exhastively check all feature
combinations to ensure they all work correctly

Additionally, this adds the needed feature gating so the crate passes
tests with `--no-default-features`
@tarcieri
tarcieri force-pushed the chacha20/use-cargo-hack-in-ci branch from f81d2da to f90219e Compare September 8, 2025 20:41
@tarcieri

tarcieri commented Sep 8, 2025

Copy link
Copy Markdown
Member Author

@newpavlov something to think about. We could also just get rid of the feature-gating and otherwise leave them as-is

@tarcieri
tarcieri merged commit 4ac5538 into master Sep 8, 2025
27 checks passed
@tarcieri
tarcieri deleted the chacha20/use-cargo-hack-in-ci branch September 8, 2025 20:49
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