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

Adds donation message to CI invocations #1154

Merged
merged 6 commits into from
Jun 8, 2024
Merged

Conversation

prabhu
Copy link
Contributor

@prabhu prabhu commented Jun 7, 2024

Active CycloneDX sponsors can pass the argument --no-banner to turn this off.

donate-panel-simple

donate-panel

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
@prabhu prabhu requested a review from setchy as a code owner June 7, 2024 10:14
@setchy
Copy link
Member

setchy commented Jun 7, 2024

Docs changes pending?

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
@prabhu
Copy link
Contributor Author

prabhu commented Jun 7, 2024

Let's not make it too easy to disable the banner :)

@prabhu prabhu requested a review from jkowalleck June 7, 2024 10:50
display.js Show resolved Hide resolved
Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>

export function printSponsorBanner(options) {
if (
process?.env?.CI &&
Copy link
Contributor Author

Choose a reason for hiding this comment

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

No idea why process is undefined in deno. Is this a regression with the latest deno?

@prabhu prabhu merged commit 5328a66 into master Jun 8, 2024
24 checks passed
@prabhu prabhu deleted the feature/sponsor-banner branch June 8, 2024 19:57
@readonlyuser1
Copy link

--noBanner

@prabhu
Copy link
Contributor Author

prabhu commented Jul 16, 2024

I think this feature has had a great success. Most users happily added --no-banner to their invocations. Total sponsorship amount raised for cdxgen remains at $0!

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

Successfully merging this pull request may close these issues.

None yet

4 participants