Skip to content

Commit

Permalink
Run osemgrep test --pro on apex/ and elixir/ too (#3478)
Browse files Browse the repository at this point in the history
test plan:
wait for green CI checks
  • Loading branch information
aryx authored Sep 23, 2024
1 parent 8ce8781 commit de1405b
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/semgrep-rules-test-develop.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,11 @@ jobs:
- uses: actions/checkout@v2
with:
path: semgrep-rules
- name: run osemgrep test --pro
run: docker run --rm -w /src -v ${GITHUB_WORKSPACE}/semgrep-rules:/src semgrep/semgrep:pro-develop semgrep test --pro .
#TODO: we can delete all the rest below and also scripts/run-tests
- name: delete directories not containing rules
run: rm -rf semgrep-rules/stats
#TODO: in theory we could run those tests by using --test --pro
# since our docker image now contains semgrep-core-pro (but
# it would require to be logged in though via a SEMGREP_APP_TOKEN)
- name: delete rules requiring Semgrep Pro
run: rm -rf semgrep-rules/apex semgrep-rules/elixir
# TODO: this takes 1m20 in CI and could be optimized by switching to osemgrep
Expand Down

0 comments on commit de1405b

Please sign in to comment.