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

update(debian): install jq on debian #6998

Merged
merged 4 commits into from
Apr 17, 2024
Merged

Conversation

ncook-hxgn
Copy link
Contributor

Closes #6997

Proposed Changes

  • Includes installation of jq on Debian so that json results files can easily be queried inside the container.
    This should help alleviate the need for those users parsing the json results file to resort to grep/cut/awk and regex, where it is easy to make mistakes or commit obtuse code.

I submit this contribution under the Apache-2.0 license.

fix Checkmarx#6997 - install jq on debian so that json results files can easily be queried inside the container. 

This should help alleviate the need for those users parsing the json results file to resort to grep/cut/awk and regex, where it is easy to make mistakes or commit obtuse code.
@github-actions github-actions bot added the community Community contribution label Apr 11, 2024
@ncook-hxgn ncook-hxgn changed the title fix #6997 - install jq on debian feature(update): install jq on debian Apr 11, 2024
@github-actions github-actions bot added the feature request Community: new feature request label Apr 11, 2024
@ncook-hxgn ncook-hxgn changed the title feature(update): install jq on debian feat(update): install jq on debian Apr 11, 2024
@ncook-hxgn
Copy link
Contributor Author

@gabriel-cx could I get a pointer? What do I need to do to get tests passing?

@gabriel-cx
Copy link
Contributor

Hi @ncook-hxgn ,

Thank you for your contribution!
I will re-run the tests, they will pass!

@gabriel-cx gabriel-cx changed the title feat(update): install jq on debian update(debian): install jq on debian Apr 16, 2024
@github-actions github-actions bot removed the feature request Community: new feature request label Apr 16, 2024
@gabriel-cx gabriel-cx merged commit f0bb9a4 into Checkmarx:master Apr 17, 2024
24 checks passed
@ncook-hxgn
Copy link
Contributor Author

@gabriel-cx <3 thank you 🥇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Community contribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(update): install jq on checkmarks/KICS:debian container
2 participants