Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #393 +/- ##
=======================================
Coverage 92.07% 92.07%
=======================================
Files 394 394
Lines 24896 24896
=======================================
Hits 22922 22922
Misses 1974 1974 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
A porting issue is created with the porting label, and that label now starts the @claude workflow, which ports the change with the port-pr skill and opens a pull request. A merged PR now gets a porting issue only when it has the needs porting label. Every new PR gets the label, so porting stays the default, and the author removes it to opt out. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
ddaspit
force-pushed
the
ddaspit/claude-port-on-issue
branch
from
October 1, 2026 18:22
b2bee4d to
9b85d3d
Compare
This comment has been minimized.
This comment has been minimized.
The mention clause did not check the event action, so adding any label to an issue that mentions @claude ran the full job before the action exited. It now skips labeled events. It also skips porting issues, which the label clause already starts, so one issue cannot start two ports. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This comment has been minimized.
This comment has been minimized.
…n-issue # Conflicts: # .github/workflows/claude.yml
This comment has been minimized.
This comment has been minimized.
Adding a needs porting label to every PR took its own pull_request_target workflow, and PRs opened before it, or ports themselves, had the wrong label. Porting is the default, so a merged PR now gets a porting issue unless it has the no porting label. The labeling workflow is removed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Public API, optional dependency, published-wheel surface, Findings: F1 addressed. Reviewed at f3df62a |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Quick summary
A new porting issue in this repo now starts Claude, which ports the change and opens a PR. A merged PR no longer gets a porting issue when it has the
no portinglabel. No library code changes.Where to look
claude.yml: theportinglabel starts the workflow through the action'slabel_trigger. An issue created with a label fires alabeledevent, so every new porting issue starts a port. The appended prompt tells Claude to use the port-pr skill. The author checks still apply. Porting issues are created with a member's token, so they pass them.claude.yml: the@claudemention clause skipslabeledevents and issues with theportinglabel. Otherwise any label on an issue that mentions@claudewould run the full job for nothing, and a porting issue that mentions@claudewould start two ports.create-porting-issue.yml: skips a merged PR with theno portinglabel. The existing check that skips a PR closing an auto-generated issue is still there, so a port does not create an issue to port it back.Porting stays the default, so nothing needs to add a label to new PRs. I renamed the
needs portinglabel I created earlier tono porting. No PR had it.Deliberately not included
The porting issues in this repo are created by
sillsdev/machine's copy ofcreate-porting-issue.yml. The opt-out there needs the same change.Validation
claude.ymlandcreate-porting-issue.ymlwith PyYAML. Both parsed.git diff --check-- clean../local_check.sh. This PR only changes workflow files.🤖 Generated with Claude Code