brave / brave/security-action

`unwrap` rule it's too noisy

Open
#527 0 comments 1 reaction 1 assignee Assigned to @thypon View on GitHub
bug
Dominant language
JavaScript
Stars
22
Forks
8
Avg merge
2d 6h
Merged PRs (30d)
40

Description

Reference: ...

# Proposed Solution

- [ ] Remove the rule
- [x] Fork and improve the rule:

- [ ] Only do it for unwrap and assume that expect means that someone at least thought about why it cannot happen.
- [ ] Ignore any file in examples/, tests/, benches/ (if not already done)
- [ ] Ignore #[test] functions
- [ ] Create a custom clippy lint, so that we can prevent the warnings from semgrep by checking/fixing them locally first

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.