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

Do not analyze language vulns unless the feature is enabled #924

Merged
merged 1 commit into from
Sep 9, 2022

Conversation

jvdm
Copy link
Contributor

@jvdm jvdm commented Sep 8, 2022

Use an empty list of analyzers if language vulnerability feature is disabled.

@jvdm jvdm changed the base branch from master to jvdm/ROX-12238/2/pull September 8, 2022 21:39
@ghost
Copy link

ghost commented Sep 8, 2022

Images are ready for the commit at c123b7a.

To use the images, use the tag 2.26-2-gc123b7aa3e.

@jvdm jvdm requested a review from RTann September 8, 2022 22:20
Copy link
Collaborator

@RTann RTann left a comment

Choose a reason for hiding this comment

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

Makes sense to me

Base automatically changed from jvdm/ROX-12238/2/pull to master September 9, 2022 14:50
@jvdm jvdm force-pushed the jvdm/12238/3/pull branch from 7b2b835 to c123b7a Compare September 9, 2022 16:31
@jvdm
Copy link
Contributor Author

jvdm commented Sep 9, 2022

/test slim-e2e-tests

@jvdm
Copy link
Contributor Author

jvdm commented Sep 9, 2022

/test e2e-tests

@jvdm jvdm merged commit a72f29d into master Sep 9, 2022
@jvdm jvdm deleted the jvdm/12238/3/pull branch September 9, 2022 18:24
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.

2 participants