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

CI: test for publishing state #181

Merged
merged 1 commit into from
Oct 2, 2023
Merged

CI: test for publishing state #181

merged 1 commit into from
Oct 2, 2023

Conversation

commial
Copy link
Contributor

@commial commial commented Oct 2, 2023

Add a CI test for cargo publish.
Notably, this command can catch weird state of internal features, or missing feature activation.

For instance, if an mla dependency enables a feature which is required by mla but not explicitly imported by it, cargo build and cargo tests will happily work. But cargo publish will spot the problem.

@commial commial added the tests Relative to the testing capabilities label Oct 2, 2023
@commial commial merged commit f872eba into master Oct 2, 2023
23 of 24 checks passed
@commial commial deleted the CI-cargo-publish branch October 2, 2023 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Relative to the testing capabilities
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant