Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

inprove sec scanning #181

Merged

Conversation

friedrichwilken
Copy link
Contributor

@friedrichwilken friedrichwilken commented Oct 24, 2023

We do not need to run tests in our code if we only bump the test scanner config.

we also do not need to scan our code in /hack since it only contains e2e tests.

@friedrichwilken friedrichwilken requested a review from a team as a code owner October 24, 2023 11:49
@kyma-bot kyma-bot added area/ci Issues or PRs related to CI related topics size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 24, 2023
@kyma-bot kyma-bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 24, 2023
@kyma-bot kyma-bot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Oct 24, 2023
@friedrichwilken friedrichwilken changed the title For tests Ignore changes to the sec-scanner-config. inprove sec scanning Oct 24, 2023
@kyma-bot kyma-bot added the lgtm Looks good to me! label Oct 24, 2023
@kyma-bot kyma-bot merged commit b066335 into kyma-project:main Oct 24, 2023
3 checks passed
@friedrichwilken friedrichwilken deleted the ignore-test-scanner-config branch February 3, 2024 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci Issues or PRs related to CI related topics lgtm Looks good to me! size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants