Skip to content

Conversation

M4tteoP
Copy link
Member

@M4tteoP M4tteoP commented Nov 19, 2024

Quantitative tests in the CI of merged PRs is failing because there is not anymore a related PR where the job can write the comment. This PR proposes to run the quantitative testing just inside the pull requests (on every commit), and not also when the PR is merged into main.

E.g. https://github.com/coreruleset/coreruleset/actions/runs/11889334419

Run thollander/actions-comment-pull-request@v3
Error: No issue/pull request in input neither in current context.

Copy link
Contributor

📊 Quantitative test results for language: eng, year: 2023, size: 10K, paranoia level: 1:
🚀 Quantitative testing did not detect new false positives

@M4tteoP M4tteoP added release:ignore Ignore for changelog release 🔧 ci/cd labels Nov 19, 2024
@M4tteoP
Copy link
Member Author

M4tteoP commented Nov 24, 2024

@fzipi Also this one is worth to be merged, right?

@fzipi
Copy link
Member

fzipi commented Nov 24, 2024

Yes!

@fzipi fzipi added this pull request to the merge queue Nov 24, 2024
Merged via the queue into main with commit 23eb444 Nov 24, 2024
8 checks passed
@fzipi fzipi deleted the quant_only_pull branch November 24, 2024 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:ignore Ignore for changelog release 🔧 ci/cd
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants