Skip to content

Conversation

@oberstet
Copy link
Contributor

Description

final round (part 5) of work: new aarch64 binary wheels, NVX), CI/CD and docs improvements before rel v25.9.1


Related Issue(s)

Closes or relates to #1677


Checklist

  • I have referenced relevant issue numbers above
  • I have performed a self-review of my code and it follows
    the style guidelines of this project
  • I have added new or used existing tests that prove my fix
    is effective or that my feature works
  • I have added necessary documentation (if appropriate) and
    updated the changelog
  • I have added an AI assistance disclosure file (required!)
    in this PR

…orkflow_run

This optimizes PR comment posting to trigger after the release workflow completes,
ensuring all 4 base workflows have finished and all artifacts are available.

Key changes:
- Trigger on workflow_run from "release" instead of pull_request_target
- Remove 5-minute polling timeout (now guaranteed to have all artifacts)
- Add GitHub API step to find wstest workflow run ID for artifact downloads
- Simplify PR context extraction using identifiers outputs
- Maintain write permissions through workflow_run context

Benefits:
- No timeout issues (waits for release completion)
- Access to all workflow artifacts (wheels, wstest, etc.)
- Future-proof (release waits for all 4 workflows)
- Cleaner architecture (release → post-comment dependency chain)
@oberstet oberstet merged commit a055a19 into crossbario:master Oct 11, 2025
27 checks passed
@oberstet oberstet deleted the rel_v25.9.1_part5 branch October 11, 2025 00:53
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.

1 participant