Transform get_pull to return full list of open PRs matching commit #244
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
EnricoMi
requested changes
Apr 3, 2022
Unit Test Results (reference) 54 files ±0 54 suites ±0 2m 38s ⏱️ +13s Results for commit 4bcd0b4. ± Comparison against base commit 71bf14b. This pull request removes 11 and adds 11 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
|
I have simplified the "hiding comments" logging message and rework the logging in general, plus adjusted the tests. |
Closed
Unit Test Results (setup-python) 54 files ±0 54 suites ±0 2m 34s ⏱️ +9s Results for commit 4bcd0b4. ± Comparison against base commit 71bf14b. This pull request removes 11 and adds 11 tests. Note that renamed tests count towards both. |
Unit Test Results (Linux) 54 files ±0 54 suites ±0 2m 34s ⏱️ +9s Results for commit 4bcd0b4. ± Comparison against base commit 71bf14b. This pull request removes 11 and adds 11 tests. Note that renamed tests count towards both. |
Unit Test Results (macOS) 54 files ±0 54 suites ±0 2m 34s ⏱️ +9s Results for commit 4bcd0b4. ± Comparison against base commit 71bf14b. This pull request removes 11 and adds 11 tests. Note that renamed tests count towards both. |
Unit Test Results (Windows) 54 files ±0 54 suites ±0 2m 34s ⏱️ +9s Results for commit 4bcd0b4. ± Comparison against base commit 71bf14b. This pull request removes 11 and adds 11 tests. Note that renamed tests count towards both. |
Unit Test Results (Test Files) 25 files ±0 4 errors 22 suites ±0 39m 12s ⏱️ ±0s For more details on these parsing errors, failures and errors, see this check. Results for commit 4bcd0b4. ± Comparison against base commit 71bf14b. |
Unit Test Results (Dockerfile) 54 files ±0 54 suites ±0 2m 34s ⏱️ +9s Results for commit 4bcd0b4. ± Comparison against base commit 71bf14b. This pull request removes 11 and adds 11 tests. Note that renamed tests count towards both. |
Unit Test Results (Docker Image) 54 files ±0 54 suites ±0 2m 34s ⏱️ +9s Results for commit 4bcd0b4. ± Comparison against base commit 71bf14b. This pull request removes 11 and adds 11 tests. Note that renamed tests count towards both. |
|
Looks like this works pretty well: https://github.com/EnricoMi/publish-unit-test-result-action/runs/5807698802?check_suite_focus=true#step:7:11 |
EnricoMi
approved these changes
Apr 3, 2022
|
Looks like everything is fine, I have no write access so I assume it's up to you to merge the PR, can you please notify me when the change gets released? |
|
Thanks for the contribution! |
|
Hi @EnricoMi , do you need to create a new release for this change to take effect? This morning the changes didn't take effect on our CI jobs. |
|
I'll release this in a few days. |
|
This has been released. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
resolves #242