| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
Also, this cleans up our pack-related integration tests. We are now testing with the most recent CLIs.
|
|
||
| # use tr to replace newlines with spaces and xargs to trim leading and trailing whitespace | ||
| RULES="$(cat javascript.sarif | jq -r '.runs[0].results[].ruleId' | sort | tr "\n" " " | xargs)" | ||
| RULES="$(cat javascript.sarif | jq -r '.runs[0].results[].ruleId' | sort | tr "\n" " " | tr "\r" " " | xargs)" |
There was a problem hiding this comment.
I think this also works (first character of string1 maps to first character of string2, and the second to the second, and so on):
tr "\r\n" " "
Sorry, something went wrong.
There was a problem hiding this comment.
Oh...didn't know tr worked that way. Thanks.
Sorry, something went wrong.
|
I'm about to make a minor change to this PR and I will ask for another review. Thanks for looking. |
Sorry, something went wrong.
This will allow the command to resolve packs with paths. Also, use a more concise version of `tr`.
|
Note that some required checks have been removed and that's why this is not mergeable. I am going to remove these as required checks and then recreate the list of required as described here: https://github.com/github/codeql-core/issues/1405#issuecomment-896190163 I'll make sure these instructions are easier to find. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Also, use newer versions of the integration test packs that allow us to run with the latest CLI.
There are a few things of note here:
Merge / deployment checklist