Skip to content

feature(ci): add audit workflow#250

Merged
cafkafk merged 5 commits into
eza-community:mainfrom
gierens:audit-workflow
Sep 11, 2023
Merged

feature(ci): add audit workflow#250
cafkafk merged 5 commits into
eza-community:mainfrom
gierens:audit-workflow

Conversation

@gierens

@gierens gierens commented Sep 11, 2023

Copy link
Copy Markdown
Member

This runs cargo deny check every night. ... This is similar but more extensive to running cargo audit locally.

PThorpe92
PThorpe92 previously approved these changes Sep 11, 2023
gierens and others added 2 commits September 11, 2023 15:37
This runs `cargo deny check` every night.

Signed-off-by: Sandro-Alessio Gierens <sandro@gierens.de>
Signed-off-by: Christina Sørensen <christina@cafkafk.com>
Signed-off-by: Christina Sørensen <christina@cafkafk.com>
Signed-off-by: Christina Sørensen <christina@cafkafk.com>

@cafkafk cafkafk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, this is cool! Added a deny.toml, and found out #227 introduced a problematic dependency, so applied the fix from the advisory db.

@cafkafk cafkafk merged commit 67f31e6 into eza-community:main Sep 11, 2023
@gierens

gierens commented Sep 11, 2023

Copy link
Copy Markdown
Member Author

Ah great, sorry for just dropping it without bothering to take care of the warnings it threw :D

@cafkafk

cafkafk commented Sep 11, 2023

Copy link
Copy Markdown
Member

Ah great, sorry for just dropping it without bothering to take care of the warnings it threw :D

Nw, worst case it would just have told us we had an issue to fix, so that's fine IMO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants