Skip to content

Commit

Permalink
Fix syntax problem in action.yml and enhanced gha workflow #2441
Browse files Browse the repository at this point in the history
  • Loading branch information
de-jcup committed Mar 22, 2024
1 parent ac55366 commit b647449
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 2 deletions.
17 changes: 16 additions & 1 deletion .github/workflows/github-action-scan.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,14 @@ jobs:

- name: Run unit tests
run: npm test

# We store git status - why? Here we see, if index.js has been changed - if so, a developer
# forgot to commit the changes - means the action cannot be used productive!
- name: Store git status
run: git status > git-status.txt


- name: Setup integration test data
- name: Define integration test setup
id : version-selector
run: |
echo "sechub_server_version=1.8.0" >> "$GITHUB_ENV"
Expand Down Expand Up @@ -76,6 +82,15 @@ jobs:
working-directory: ./github-actions/scan/__test__/integrationtest/
run: ./05-stop.sh $sechub_server_port $pds_port

# ------------------------------------ Archive git status-------------------
- name: Archive git status
if: always()
uses: actions/upload-artifact@v3
with:
name: git-status
path: ./git-status.txt
retention-days: 14

# ------------------------------------ Archive runtime logs-------------------
- name: Archive runtime logs
if: always()
Expand Down
2 changes: 1 addition & 1 deletion github-actions/scan/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ inputs:
required: false
default: '1.4.0'
scan-types:
description: 'Scan types to use for generated sechub configuration file (ignored in the case of a custom configuration file). Supported types are: codeScan, licenseScan and secretScan. Can be combined by ,'
description: 'Scan types to use for generated sechub configuration file (ignored in the case of a custom configuration file). Supported types are: codeScan, licenseScan and secretScan. Can be combined by ,'
required: false
default: 'codeScan'
content-type:
Expand Down

0 comments on commit b647449

Please sign in to comment.