Skip to content

ci: Replace actions-rs/audit-check with rustsec fork#294

Merged
FabianLars merged 3 commits intodevfrom
ci/audit
Apr 3, 2023
Merged

ci: Replace actions-rs/audit-check with rustsec fork#294
FabianLars merged 3 commits intodevfrom
ci/audit

Conversation

@FabianLars
Copy link
Copy Markdown
Member

They finally decided to maintain it for now and recently released an update. rustsec is the same org that hosts the advisory database so imo it should be fairly trustworthy.

This PR is a follow-up of #45 where we replaced all other actions-rs actions.

@FabianLars FabianLars requested a review from a team as a code owner March 31, 2023 15:40
@FabianLars FabianLars requested a review from Beanow March 31, 2023 15:41
@FabianLars
Copy link
Copy Markdown
Member Author

@Beanow Requesting your review because you participated in #45.

@Beanow
Copy link
Copy Markdown
Member

Beanow commented Mar 31, 2023

I think you're referring to rustsec/audit-check#2 (comment)

Seems like we should report a bug though:

Warning: Unknown warning kind unsound found, please, file a bug

@FabianLars
Copy link
Copy Markdown
Member Author

Alrighty, tried to fix it myself but i don't understand the action at all 😅

@FabianLars
Copy link
Copy Markdown
Member Author

rustsec/audit-check#8

@FabianLars FabianLars merged commit cea1d4f into dev Apr 3, 2023
@FabianLars FabianLars deleted the ci/audit branch April 3, 2023 09:39
OrIOg pushed a commit to OrIOg/plugins-workspace that referenced this pull request Jun 25, 2023
* ci: Replace actions-rs/audit-check with rustsec fork

* Update audit-rust.yml

* Revert "Update audit-rust.yml"
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