Skip to content

Conversation

gitulisca
Copy link
Contributor

Re-creating as a replacement for PR99 which was approved but couldn't be merged due to a broken CodeQL workflow unrelated to the original PR; creating a new PR was chosen in favor of fixing the existing one with an empty commit

This change will add an action input parameter called correlator which allows for specifying a custom snapshot correlator value.

Using a custom snapshot correlator value is useful when the action is used in a workflow that has a matrix and when it is used on a reusable workflow that targets different paths in a mono repo.

It also provides for extra flexibility that may be required in other use cases when the job id isn't the most appropriate choice for the correlator value .

@gitulisca gitulisca requested review from a team as code owners April 7, 2025 01:58
@gitulisca gitulisca requested review from GeekMasher and adrienpessu and removed request for a team April 7, 2025 01:58
@GeekMasher
Copy link
Contributor

This looks good to me @gitulisca but it seems the CI tests are failing. Let me know when you've updated the tests / verified the issue

@gitulisca
Copy link
Contributor Author

gitulisca commented Apr 7, 2025

This looks good to me @gitulisca but it seems the CI tests are failing. Let me know when you've updated the tests / verified the issue

Could this possibly be a flaky?
I had to fix a previous test issue by increasing the jest timeout for one of the tests.
It doesn't seem to behave in a deterministic fashion as different Runners will take different times to complete that step.

@GeekMasher
Copy link
Contributor

@gitulisca that's old. Absolutely it seems to be flakey so let's get it merged

@GeekMasher GeekMasher merged commit 4c4b8a2 into advanced-security:main Apr 7, 2025
2 checks passed
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.

3 participants