Conversation
This adds the Claude Code workflows from machine.py. The @claude workflow can now open PRs from issues, and a porting issue starts a port with the new port-pr skill. The code review runs on every push and follows up on its earlier findings instead of posting them again. A merged PR now gets a porting issue unless it has the no porting label. Ports sillsdev/machine.py#377, #384, #389, #391, #392, #394 and #393. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
ddaspit
added
the
no porting
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #535 +/- ##
=======================================
Coverage 74.34% 74.34%
=======================================
Files 456 456
Lines 38261 38261
Branches 5242 5242
=======================================
Hits 28445 28445
Misses 8666 8666
Partials 1150 1150 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
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
2 participants
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
This adds the Claude Code workflows and skill updates from machine.py. A porting issue now starts a port that opens its own PR. No library code changes.
Where to look
.github/workflows/claude.ymlbuilds sentencepiece4c and restores likeci.yml, and clones machine.py to../machine.pyfor the port-pr skill. It has not run yet..github/workflows/create-porting-issue.ymlnow skips PRs with theno portinglabel. This repo does not have that label yet..claude/skills/port-pr/SKILL.mdis machine.py's skill reversed, porting Python to C#..claude/skills/pr-review/SKILL.mdfollows up on earlier findings, numbers them, and uses the Reviewable keywords.Deliberately not included
CLAUDE_CODE_OAUTH_TOKENsecret and the Claude GitHub app are not set up on this repo yet. The two Claude workflows will fail until they are.Validation
./local_check.sh --agent-strict-- passed, 1083 tests passed and 3 skipped. The hygiene scan found no in-scope files.git diff --check origin/master...HEAD-- cleanIssue / porting context
Ports sillsdev/machine.py#377, sillsdev/machine.py#384, sillsdev/machine.py#389, sillsdev/machine.py#391, sillsdev/machine.py#392, sillsdev/machine.py#394 and sillsdev/machine.py#393. #393 is still open.
🤖 Generated with Claude Code
This change is