tinezivic commented on PR #49797: URL: https://github.com/apache/arrow/pull/49797#issuecomment-4284028505
The `Source Release and Merge Script` CI failures are unrelated to the C++ changes in this PR. **Root cause:** Commit apache/arrow@c7cb411 (`GH-49793: [R] Update NEWS.md for 24.0.0`, merged April 19) updated `r/NEWS.md` to start with `# arrow 24.0.0.9000`, while `r/DESCRIPTION` still has `Version: 23.0.1.9000`. This inconsistency breaks `01-prepare-test.rb::test_version_pre_tag` and `post-10-bump-versions-test.rb` because: - `detect_versions` reads `@previous_version = "23.0.1"` from `r/DESCRIPTION` - The test expects `r/NEWS.md` to change from `# arrow 23.0.1.9000` → `# arrow 24.0.0` - But after the merge, `r/NEWS.md` has `# arrow 24.0.0.9000` at the top → actual change is `24.0.0.9000` → `24.0.0`, which doesn't match the expected diff This is a project-wide regression affecting all PRs whose CI runs after that commit (e.g. #49801, #49804, #49806 also fail). PR #49798 passes only because its last CI run was ~21 hours before `c7cb411` was merged. The fix would be to either update `r/DESCRIPTION` to `Version: 24.0.0.9000` (consistent with `r/NEWS.md`) or revert/adjust `c7cb411`, which should be done in a separate upstream PR. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
