Skip to content

Conversation

@Lytol
Copy link
Contributor

@Lytol Lytol commented Jan 12, 2023

  • Do not include code/static analysis
  • Use monitor instead of test
  • Target the setup.py
  • Do not scan the requirements.txt (dev dependencies)

Closes #333

@Lytol Lytol requested a review from mmarchetti January 12, 2023 23:13
@Lytol Lytol force-pushed the snyk-integration branch 2 times, most recently from 8ee3723 to 1ebdc16 Compare January 17, 2023 18:02
- Do not include code/static analysis
- Use `monitor` instead of `test`
- Target the setup.py for release and requirements.txt for dev
@Lytol Lytol merged commit ac9860b into master Jan 17, 2023
@Lytol Lytol deleted the snyk-integration branch January 17, 2023 18:22
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.

Add Snyk monitoring via GitHub actions

3 participants