Skip to content

Commit

Permalink
fix: use the correct exclude rule for semgrep
Browse files Browse the repository at this point in the history
  • Loading branch information
KyleKing committed Jul 17, 2023
1 parent 211cf7f commit a18b17c
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
2 changes: 1 addition & 1 deletion calcipy/tasks/lint.py
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ def security(ctx: Context) -> None:
'--config=r/python',
'--config=r/terraform',
'--config=r/yaml',
'--exclude-rule=third-party-action-not-pinned-to-commit-sha',
'--exclude-rule=yaml.github-actions.security.third-party-action-not-pinned-to-commit-sha.third-party-action-not-pinned-to-commit-sha',
])
logger.text('Note: Selectively override semgrep with "# nosem"', is_header=True)
run(ctx, f'{python_dir()}/semgrep ci --autofix {semgrep_configs}')
Expand Down
4 changes: 4 additions & 0 deletions docs/docs/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,10 @@
- support pydantic v2 serialization
- skip SemGrep rule to pin GitHub Actions to commit IDs

### Refactor

- resolve local test suite problems

## 1.4.1 (2023-07-07)

### Fix
Expand Down
2 changes: 1 addition & 1 deletion tests/tasks/test_lint.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
'--config=r/python',
'--config=r/terraform',
'--config=r/yaml',
'--exclude-rule=third-party-action-not-pinned-to-commit-sha',
'--exclude-rule=yaml.github-actions.security.third-party-action-not-pinned-to-commit-sha.third-party-action-not-pinned-to-commit-sha',
]),
]),
(pre_commit, {}, [
Expand Down

0 comments on commit a18b17c

Please sign in to comment.