You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by "Xuanwo (via GitHub)" <gi...@apache.org> on 2026/09/18 16:46:12 UTC
[PR] fix(release): validate declarations and published state correctly [opendal]
Xuanwo opened a new pull request, #8322:
URL: https://github.com/apache/opendal/pull/8322
# Which issue does this PR close?
Related to #8284 and #8299.
# Rationale for this change
Two validation assumptions block release automation:
- [Weekly preparation](https://github.com/apache/opendal/actions/runs/35293207006) rejects the `None.` Breaking changes section in #8299 because it accepts `None` but not a trailing period.
- [Publication follow-up](https://github.com/apache/opendal/actions/runs/35303527131) fails with `ATR revision is missing` for 0.59.2-rc.2 even though ATR reports `phase: release` and a completed publication timestamp. Its `latest_revision_number` is null, but the notification code still requires a candidate revision.
# What changes are included in this PR?
Normalize trailing periods in explicit non-breaking declarations while retaining affected-package and migration requirements for PRs labeled `breaking-changes`.
Require a revision only before the ATR release phase. Published-state notices link to the release and describe follow-up without generating candidate voting commands or a missing revision. The publication-resume test now exercises the real notification function with a null revision and verifies that cleanup remains reachable without repeating the ATR announcement.
The fixes were checked against the actual #8299 body and ATR response. The complete declaration scan from v0.59.2 to `2698e90a1131a477fc3382465f1c28bd2202395e` passed for all 19 PRs. This validates the declaration stage, not a complete new candidate build or deployed publication run.
# Are there any user-facing changes?
Compatible PRs declaring `None.` no longer block weekly preparation. Already-published releases can continue optional distribution and cleanup without a candidate revision. Existing release artifacts and tags are unchanged.
# Breaking changes
# AI Usage Statement
- Harness: Codex desktop.
- Model: gpt-6-astra.
- Effort: medium.
- Role: Investigated the failed workflows and live ATR/PR data, implemented and tested the fixes, and prepared this PR. The updated automation has not yet run on GitHub.
--
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: commits-unsubscribe@opendal.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(release): validate declarations and published state correctly [opendal]
Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo merged PR #8322:
URL: https://github.com/apache/opendal/pull/8322
--
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: commits-unsubscribe@opendal.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org