Skip to content

Conversation

NucleonGodX
Copy link
Collaborator

fixes #22

@NucleonGodX NucleonGodX marked this pull request as ready for review August 17, 2025 14:17
@NucleonGodX
Copy link
Collaborator Author

Hey @tdruez I think for now I'd like this PR to only refine github actions method, review it when you have time and then I will refine the PR #19 to support azurepipelines and similarly the next one #21 for jenkins?

action.yml Outdated
output-formats:
description: "Output formats"
default: "json xlsx spdx cyclonedx"
default: "json xlsx spdx cyclonedx"
Copy link
Contributor

Choose a reason for hiding this comment

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

Leftover?

action.yml Outdated
Comment on lines 54 to 58
- name: Download ScanCode Pipeline Script
shell: bash
run: |
echo "SECRET_KEY=$(openssl rand -base64 32)" >> $GITHUB_ENV
echo "SCANCODEIO_DB_NAME=scancodeio" >> $GITHUB_ENV
echo "SCANCODEIO_DB_USER=scancodeio" >> $GITHUB_ENV
echo "SCANCODEIO_DB_PASSWORD=scancodeio" >> $GITHUB_ENV
curl -o scancode_pipeline.py https://raw.githubusercontent.com/NucleonGodX/scancode-action/improve-structure/scripts/scancode_pipeline.py
chmod +x scancode_pipeline.py
Copy link
Contributor

Choose a reason for hiding this comment

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

Is the local script of the repo scripts/scancode_pipeline.py already available?
We do not want o have to download anything external in any cases.

@@ -0,0 +1,354 @@
"""
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add the usual AboutCode notice.

@NucleonGodX
Copy link
Collaborator Author

hey @tdruez can you please review this pr when you have time.

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

Successfully merging this pull request may close these issues.

Add a mechanism to eliminate redundant Bash code across CI providers
2 participants