Skip to content

fix: silence the warnings cargo check --all-targets reports - #2369

Open
zac-scheiwe wants to merge 1 commit into
pereq-dev:mainfrom
zac-scheiwe:fix-lints
Open

zac-scheiwe wants to merge 1 commit into
pereq-dev:mainfrom
zac-scheiwe:fix-lints

Conversation

@zac-scheiwe

Copy link
Copy Markdown
Contributor

cargo check --all-targets currently reports three warnings. This PR fixes them; there is no behaviour change.

  • hax-adt-into: AdtInto emits #[cfg(feature = "rustc")] into the crate that derives it. This crate's own test derives it, but hax-adt-into has no rustc feature, so unexpected_cfgs fires. Declare the value with check-cfg under [lints.rust].
  • hax-adt-into tests: make the mirrored module pub so its example type isn't reported as dead code.
  • hax-rust-engine: the crate enables unused_crate_dependencies, and hax-lib is a dev-dependency used only for doc links. Mark it used in test builds with #[cfg(test)] use hax_lib as _;.

[skip changelog]

- hax-adt-into: declare the `rustc` feature cfg that `AdtInto` emits,
  and make the test's `mirrored` module public so its example type
  is not reported as dead code.
- hax-rust-engine: mark the `hax-lib` dev-dependency as used in test
  builds; it is only there for documentation links.

This branch has not been deployed

No deployments
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.

1 participant