Skip to content

fix: publish-suppressions action trigger#8516

Merged
jeremylong merged 1 commit into
mainfrom
scratch/publish-suppression-trigger
May 19, 2026
Merged

fix: publish-suppressions action trigger#8516
jeremylong merged 1 commit into
mainfrom
scratch/publish-suppression-trigger

Conversation

@jeremylong
Copy link
Copy Markdown
Collaborator

resolves #8512

@jeremylong jeremylong added this to the 13.0.0 milestone May 19, 2026
@jeremylong jeremylong merged commit f03b7dc into main May 19, 2026
15 checks passed
@jeremylong jeremylong deleted the scratch/publish-suppression-trigger branch May 19, 2026 11:43
@chadlwilson
Copy link
Copy Markdown
Collaborator

chadlwilson commented May 19, 2026

I think this will double-trigger/double-publish since the approvals workflow pushes to the branch and then triggers publish directly?

I was part way through unpicking that, but was a bit confused about the logic 😅

@chadlwilson
Copy link
Copy Markdown
Collaborator

Hmm, it didn't seem to double-trigger after an FP approval just now, so perhaps GH have some protection in place for internal actions pushes. Maybe my fears were misplaced 👍🏻

@chadlwilson
Copy link
Copy Markdown
Collaborator

chadlwilson commented May 20, 2026

Unfortunately this doesn’t seem to be working - merge of #8522 didn’t trigger anything. That’s probably also why it didn’t double trigger.

https://github.com/dependency-check/DependencyCheck/actions?query=workflow%3A%22Publish%20Suppressions%22

I suspect this is because push events run within, and use workflow definition of, the relevant branch’s context, yet the generatedSuppressions branch is not a proper branch, and has no workflows. Workflow dispatch makes you select the context, which is why it works there.

I don’t think it’s a good idea to merge the workflows over to that branch as they will potentially become out of date, or hard to maintain. I’ll have a think about how we might be able to do this maintainable with minimal changes?

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.

Improve publish-suppressions action trigger

2 participants