Skip to content

ci: adjust maven publishing wait until condition#738

Merged
ricellis merged 1 commit intomainfrom
s1227-maven-central-state
Sep 4, 2025
Merged

ci: adjust maven publishing wait until condition#738
ricellis merged 1 commit intomainfrom
s1227-maven-central-state

Conversation

@ricellis
Copy link
Copy Markdown
Member

@ricellis ricellis commented Sep 4, 2025

PR summary

Adjust maven publishing wait until condition

Fixes: s1227

PR Checklist

Please make sure that your PR fulfills the following requirements:

  • The commit message follows the
    Angular Commit Message Guidelines.
  • Tests for the changes have been added (for bug fixes / features) - build
  • Docs have been added / updated (for bug fixes / features) - build

PR Type

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • New tests
  • Build/CI related changes
  • Documentation content changes
  • Other (please describe)

What is the current behavior?

Waits up to half an hour for library to publish to Maven Central, polling for state every 5 seconds.

What is the new behavior?

Waits up to an hour for library to validate, polling for state every 15 seconds.

The sync from Central Portal to Maven Central timed out during the last release. Since we always check that the release has reached Maven Central before marking the release process completed anyway we can avoid potential future timeouts by waiting only for the validated state. In validated state we know the upload is good and meets the requirements of Maven Central. The sync from Central Portal is outside of our control anyway, so there is little point in failing our build if it doesn't happen.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@ricellis ricellis requested a review from a team as a code owner September 4, 2025 10:05
@ricellis ricellis self-assigned this Sep 4, 2025
@ricellis ricellis merged commit 13e9435 into main Sep 4, 2025
9 checks passed
@ricellis ricellis deleted the s1227-maven-central-state branch September 4, 2025 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants