Skip to content

ci: trigger PackageDistro from PR comments#6379

Merged
fingolfin merged 1 commit into
masterfrom
codex/pkgdist-pr-trigger
May 18, 2026
Merged

ci: trigger PackageDistro from PR comments#6379
fingolfin merged 1 commit into
masterfrom
codex/pkgdist-pr-trigger

Conversation

@fingolfin
Copy link
Copy Markdown
Member

Add an issue_comment workflow for pull requests that accepts the exact bot command, checks collaborator permissions, and dispatches the PackageDistro workflow for the PR head.

The workflow also keeps a bot comment on the PR up to date so rejected requests get a clear reply and accepted requests are replaced by the later PackageDistro result.

AI assistance: Codex implemented the workflow and
verification updates.


Disclaimer: I have not yet reviewed this PR myself at all. So if you don't like being exposed to unvetted AI slop, better don't look yet. But I don't want to risk forgetting this start, so I am putting it out here.

This PR here requires gap-system/PackageDistro#1354 to be really useful.

@fingolfin fingolfin added release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes topic: infrastructure labels May 7, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.73%. Comparing base (248817b) to head (5f67990).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6379      +/-   ##
==========================================
- Coverage   78.73%   78.73%   -0.01%     
==========================================
  Files         685      685              
  Lines      293358   293358              
  Branches     8658     8658              
==========================================
- Hits       230988   230985       -3     
  Misses      60552    60552              
- Partials     1818     1821       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@fingolfin fingolfin force-pushed the codex/pkgdist-pr-trigger branch from 2a7f4f3 to 018ae0a Compare May 18, 2026 08:43
Add an issue_comment workflow for pull requests that
accepts the exact bot command, checks collaborator
permissions, and dispatches the PackageDistro workflow
for the PR head.

The workflow also keeps a bot comment on the PR up to
date so rejected requests get a clear reply and accepted
requests are replaced by the later PackageDistro result.

AI assistance: Codex implemented the workflow and
verification updates.

Co-authored-by: Codex <codex@openai.com>
@fingolfin fingolfin force-pushed the codex/pkgdist-pr-trigger branch from 018ae0a to 5f67990 Compare May 18, 2026 08:55
@fingolfin fingolfin marked this pull request as ready for review May 18, 2026 08:55
@fingolfin
Copy link
Copy Markdown
Member Author

I am not sure how well this will work, but let's give it a spin...

@fingolfin fingolfin merged commit f261316 into master May 18, 2026
24 of 26 checks passed
@fingolfin fingolfin deleted the codex/pkgdist-pr-trigger branch May 18, 2026 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes topic: infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant