-
Notifications
You must be signed in to change notification settings - Fork 4
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
Adding Whitesource Scan to Build Workflow #64
Conversation
@@ -23,7 +23,7 @@ jobs: | |||
contents: read | |||
packages: write | |||
steps: | |||
- uses: actions/checkout@v2 | |||
- uses: actions/checkout@v3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
upversion the plugin, to address this warning: actions/checkout#1047
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
.github/workflows/build.yaml
Outdated
@@ -37,6 +37,16 @@ jobs: | |||
export SKIP_FLAGS_NON_UNIT_TESTS="-Dcheckstyle.skip -Dpmd.skip -Dcpd.skip -Dfindbugs.skip -Dspotbugs.skip" | |||
echo "SKIP_FLAGS_NON_UNIT_TESTS=$SKIP_FLAGS_NON_UNIT_TESTS" >> $GITHUB_ENV | |||
echo "SKIP_FLAGS_ALL_TESTS=$SKIP_FLAGS_NON_UNIT_TESTS -Dmaven.test.skip=true" >> $GITHUB_ENV | |||
if [[ $GITHUB_EVENT_NAME != "pull_request" && $GITHUB_REF_NAME=="main" ]]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trying to remember here the case where GITHUB_EVENT_NAME==pull_request
and GITHUB_REF_NAME=="main"
. Why did we have to specify != pull_request
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, I also realized that $GITHUB_EVENT_NAME != "pull_request"
is not necessary, for PRs GITHUB_REF_NAME
vaule is pr_number/merge
;
so going to drop that!
What is the purpose of this change?
Adding Whitesource Scan to Build Workflow
How was this change implemented?
Whitesource scan step added to the build workflow, which does:
wss-unified-agent.jar
scan againsttarget/lib
folderWS_EXCLUDES
How was this change tested?