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

Add cargo deny config and audit workflow #156

Merged
merged 6 commits into from
Jun 10, 2024

Conversation

gierens
Copy link
Contributor

@gierens gierens commented Apr 6, 2024

This adds the config file for cargo deny which automatically checks for things
like advisories, licenses, bans, ... and a corresponding audit workflow that
runs on every push changing dependencies and every night.

Signed-off-by: Sandro-Alessio Gierens sandro@gierens.de

@gierens gierens force-pushed the audit-workflow branch 3 times, most recently from 28db234 to dd1301e Compare April 7, 2024 06:42
@gierens
Copy link
Contributor Author

gierens commented Apr 7, 2024

@dtantsur The functional tests are failing because the setup-rust action just vanished from Github. #157 tries to replace it with an alternative.

@dtantsur
Copy link
Owner

#157 merged, could you rebase all your PRs?

Signed-off-by: Sandro-Alessio Gierens <sandro@gierens.de>
…push

Signed-off-by: Sandro-Alessio Gierens <sandro@gierens.de>
Signed-off-by: Sandro-Alessio Gierens <sandro@gierens.de>
Signed-off-by: Sandro-Alessio Gierens <sandro@gierens.de>
Signed-off-by: Sandro-Alessio Gierens <sandro@gierens.de>
@gierens
Copy link
Contributor Author

gierens commented Apr 14, 2024

@dtantsur I rebased it.

@dtantsur
Copy link
Owner

Wow, interesting, I did not know about cargo-deny. TIL! https://github.com/dtantsur/rust-openstack/actions/runs/8679800639/job/23799281981?pr=156 shows a lot of deprecation warnings, could you check if any of these are important?

Signed-off-by: Sandro-Alessio Gierens <sandro@gierens.de>
@gierens
Copy link
Contributor Author

gierens commented May 8, 2024

I updated the deny.toml according to the new template, the deprecation warnings are gone now.

@gierens
Copy link
Contributor Author

gierens commented May 9, 2024

The failing functional test on zed seems to be unrelated.

@dtantsur dtantsur merged commit c2ab066 into dtantsur:master Jun 10, 2024
15 of 16 checks passed
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