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.
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
🌱 Recreate checks always to keep them in the upper check suite #35
Conversation
…tack so that they are showed in the checks section Signed-off-by: Adrian Orive <adrian.orive.oneca@gmail.com>
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Adirio The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Proof of the issue that I'm tackling can be found comparing this PR, where only the first open action was issued, and therefore the checks section is complete. |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-contributor-experience at kubernetes/community. |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-contributor-experience at kubernetes/community. |
Rotten issues close after 30d of inactivity. Send feedback to sig-contributor-experience at kubernetes/community. |
@fejta-bot: Closed this PR. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
As GitHub CheckRuns and CheckSuite API doesn't allow to bind Check Runs to Check Suites, reusing a CheckRun for any action is not a valid approach. Any action (open, reopen, edit, sync) creates a new Check Suite which is the one shown in the PR. If we reuse a previous Check Run (which we can't bind to this new Check Suite) it will be hidden in the previous Check Suite and, while its results will be updated and properly used, the check sections will not show the corresponding message.
This PR recreates checks always to keep them in the upper Check Suite of the stack so that they are showed in the checks section of a PR, unless until a way of binding Check Runs to Check Suites is provided by GitHub.