Skip to content

label for contracts artifact release workflow to run before merge#642

Open
vicentevieytes wants to merge 2 commits intomainfrom
vv/label-to-trigger-publish-contracts-workflow
Open

label for contracts artifact release workflow to run before merge#642
vicentevieytes wants to merge 2 commits intomainfrom
vv/label-to-trigger-publish-contracts-workflow

Conversation

@vicentevieytes
Copy link
Collaborator

No description provided.

@vicentevieytes vicentevieytes requested a review from a team as a code owner March 6, 2026 17:20
Copilot AI review requested due to automatic review settings March 6, 2026 17:20
@github-actions
Copy link

github-actions bot commented Mar 6, 2026

👋 vicentevieytes, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the TON contracts compiled-artifacts publishing workflow so it can be explicitly opted into on pull requests via a label, while preserving existing behavior for main pushes and manual dispatch runs.

Changes:

  • Add pull_request event types to include labeled (and explicitly list existing default PR actions).
  • Gate the publish job on PRs behind the build-publish-contracts label (while still requiring relevant contract/workflow changes or a manual dispatch).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 11 to +12
pull_request:
types: [labeled, opened, synchronize, reopened]
Copy link

Copilot AI Mar 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With pull_request triggering on opened/synchronize/reopened (plus labeled), the workflow will still run the changes job for PRs that don’t have the build-publish-contracts label, even though publish will be skipped. If the intent is to only spend CI time when the label is present, consider adding a similar label-gated if: to the changes job (or otherwise gating early), so the workflow no-ops quickly on unlabeled PR events.

Copilot uses AI. Check for mistakes.
@vicentevieytes vicentevieytes changed the title label to contracts artifact release workflow to run before merge label for contracts artifact release workflow to run before merge Mar 6, 2026
@vicentevieytes vicentevieytes enabled auto-merge (squash) March 6, 2026 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants