Skip to content
This repository has been archived by the owner on Oct 26, 2023. It is now read-only.

fix(code-analysis): disabled code analysis #116

Merged
merged 10 commits into from
Oct 4, 2022
Merged

fix(code-analysis): disabled code analysis #116

merged 10 commits into from
Oct 4, 2022

Conversation

gwnlng
Copy link
Contributor

@gwnlng gwnlng commented Aug 25, 2022

disable code argument with off-only option

gwnlng added 2 commits August 25, 2022 21:08
disable code argument with off-only option
gwnlng added 4 commits August 31, 2022 16:02
snyk security scans only at push with secret token use
snyk security scans only at push with secret token use at build-test-monitor
name: snyk_security

# Controls when the workflow will run
on:
Copy link
Contributor

@scott-es scott-es Sep 24, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's still do the snyk test on PRs if they are not from a fork. its a little more work but still most of the repo changes will be from internal PRs and will still be useful to have this in the PR check.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

included snyk* jobs conditional checks to internal PRs from this base repo

@@ -1,4 +1,17 @@
version: 2.1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we just delete this file? we don't want to have anything related to circle CI left

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

deleted

snyk security scans trigger on PR to master from same repo branches
snyk security jobs trigger on PR to master from same repo branches
snyk security jobs trigger on PR to master from same repo branches
snyk security jobs trigger on PR to master from same repo branches
@gwnlng gwnlng merged commit 594c8f5 into snyk-labs:master Oct 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants