You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hugegraph.apache.org by "contrueCT (via GitHub)" <gi...@apache.org> on 2026/10/09 12:00:01 UTC
[PR] fix(ci): reject obsolete PR targets [hugegraph]
contrueCT opened a new pull request, #3297:
URL: https://github.com/apache/hugegraph/pull/3297
## Purpose of the PR
Fixes #3296. A queued PR run currently accepts its old merge after same-repository retargeting or a target force-push to unrelated history. Plans and final gates now reject these stale targets while allowing normal target-branch advancement.
## Main Changes
- Record and verify the target branch name in plans and result reports.
- Check the tested base against the live target with the repository compare API; commits added after checkout need not be present locally.
- Keep conservative selection and the existing Memory metadata-outage policy.
## Verifying these changes
- [ ] Trivial rework / code cleanup without any test coverage. (No Need)
- [x] Covered by regression tests: 56 Python tests pass, including real Git fixtures for retargeting, unrelated force-push, ordinary advancement and all three final gates.
- [x] Java 17 whole-repository clean compilation, editorconfig formatting and `git diff --check` pass. A live repository comparison confirms the API status/merge-base fields used by the guard.
## Does this PR potentially affect the following parts?
- [ ] Dependencies
- [ ] Modify configurations
- [ ] The public API
- [x] Other affects: CI plan freshness and result reporting
- [ ] Nope
## Documentation Status
- [ ] `Doc - TODO`
- [x] `Doc - Done`: `docs/ci.md` documents target-ref binding, ancestry and outage handling.
- [ ] `Doc - No Need`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hugegraph.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@hugegraph.apache.org
For additional commands, e-mail: issues-help@hugegraph.apache.org
Re: [PR] fix(ci): reject obsolete PR targets [hugegraph]
Posted by "contrueCT (via GitHub)" <gi...@apache.org>.
contrueCT closed pull request #3297: fix(ci): reject obsolete PR targets
URL: https://github.com/apache/hugegraph/pull/3297
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hugegraph.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@hugegraph.apache.org
For additional commands, e-mail: issues-help@hugegraph.apache.org
Re: [PR] fix(ci): reject obsolete PR targets [hugegraph]
Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.
codecov[bot] commented on PR #3297:
URL: https://github.com/apache/hugegraph/pull/3297#issuecomment-6080946661
## [Codecov](https://app.codecov.io/gh/apache/hugegraph/pull/3297?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
:white_check_mark: All modified and coverable lines are covered by tests.
:white_check_mark: Project coverage is 33.55%. Comparing base ([`6885519`](https://app.codecov.io/gh/apache/hugegraph/commit/68855199031d5801edb4fe41b2bacbacffe8fe68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`ca90d74`](https://app.codecov.io/gh/apache/hugegraph/commit/ca90d74db840d5af58af62c4033b6276db23a0c0?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #3297 +/- ##
============================================
- Coverage 37.05% 33.55% -3.50%
+ Complexity 6006 3775 -2231
============================================
Files 757 492 -265
Lines 65152 45887 -19265
Branches 8604 6499 -2105
============================================
- Hits 24141 15399 -8742
+ Misses 38322 28499 -9823
+ Partials 2689 1989 -700
```
</details>
[:umbrella: View full report in Codecov by Harness](https://app.codecov.io/gh/apache/hugegraph/pull/3297?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
:loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
<details><summary> :rocket: New features to boost your workflow: </summary>
- :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems.
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hugegraph.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@hugegraph.apache.org
For additional commands, e-mail: issues-help@hugegraph.apache.org
Re: [PR] fix(ci): reject obsolete PR targets [hugegraph]
Posted by "contrueCT (via GitHub)" <gi...@apache.org>.
contrueCT commented on PR #3297:
URL: https://github.com/apache/hugegraph/pull/3297#issuecomment-6095665941
Closed after reconsidering the need for this change. A CI run validates the commit or test merge commit associated with that run. GitHub’s status-check and branch-protection rules determine whether those results satisfy the current PR’s merge requirements.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hugegraph.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@hugegraph.apache.org
For additional commands, e-mail: issues-help@hugegraph.apache.org