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.

Sign up for GitHub

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

Jump to bottom

[CI] Create unique OWASP dependency check reports for each branch #13601

Merged
merged 1 commit into from
Jan 4, 2022
Merged

[CI] Create unique OWASP dependency check reports for each branch #13601

merged 1 commit into from
Jan 4, 2022

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Jan 4, 2022

Motivation

  • currently the uploaded reports for different branches get overwritten since a common name is used

Modifications

  • use branch name as part of uploaded filename

@lhotari
[CI] Create unique OWASP dependency check reports for each branch
3b64ea7 
- currently the uploaded reports for different branches
  get overwritten since a common name is used
@lhotari lhotari added the doc-not-needed Your PR changes do not impact docs label Jan 4, 2022
@lhotari lhotari added this to the 2.10.0 milestone Jan 4, 2022
@lhotari lhotari self-assigned this Jan 4, 2022
nicoloboschi
nicoloboschi approved these changes Jan 4, 2022
View reviewed changes
Copy link
Contributor

@nicoloboschi nicoloboschi left a comment

Choose a reason for hiding this comment

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

+1

michaeljmarshall
michaeljmarshall approved these changes Jan 4, 2022
View reviewed changes
Copy link
Member

@michaeljmarshall michaeljmarshall left a comment

Choose a reason for hiding this comment

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

LGTM

@lhotari lhotari merged commit 561cde6 into apache:master Jan 4, 2022
wuzhanpeng pushed a commit to wuzhanpeng/pulsar that referenced this pull request Jan 5, 2022
@lhotari
[CI] Create unique OWASP dependency check reports for each branch (ap…
abc802c 
…ache#13601)

- currently the uploaded reports for different branches
  get overwritten since a common name is used
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nicoloboschi nicoloboschi nicoloboschi approved these changes

@michaeljmarshall michaeljmarshall michaeljmarshall approved these changes

@eolivelli eolivelli Awaiting requested review from eolivelli

Assignees

@lhotari lhotari

Labels
doc-not-needed Your PR changes do not impact docs
Projects
None yet
Milestone
2.10.0
Development

Successfully merging this pull request may close these issues.
3 participants
@lhotari @nicoloboschi @michaeljmarshall