Repository navigation
Conversation
GitHub Actions Integration
✅ All 1 passed. |
|
This PR has too many changes. Please make sure you are NOT addressing multiple issues with one PR. |
|
We are choosing #47 (GitHub Script with runtime npm installation) for this action. The work is primarily GitHub API orchestration. GitHub Script already supplies the authenticated client, pagination, workflow context, outputs, and failure reporting. After removing bundling from #47, its plain JavaScript implementation has fewer integration components to maintain. The Python/uv alternative additionally needs Actions environment/output plumbing and YAML compatibility code. Both alternatives passed their behavior tests and native AMD64/ARM64 smoke tests. #48 established a useful comparison and helped expand the shared behavioral contract. We will continue the requested interface refinements in #47 and close this alternative in favor of that implementation. |
👋 Background
Provide a Python/uv alternative to #47 with the same action path, policy inputs, outputs, and reusable-workflow integration. Both PRs target main independently; choose one implementation, not both.
⚙️ Description
.github/actions/github.pr.size-labelerusinguv run --no-config --locked --script size_labeler.py, uv 0.13.2, and Python 3.12. PEP 723 metadata and the adjacent lock file replace a package project and generated build artifacts..github/size-labeler.yaml, then.github/size-labeler.yml, or useconfig_file. Read only from the PR base SHA. File keys override matching policy inputs, while auth/API URL/PR number/config path stay invocation inputs.Validation: 77 local tests passed, including the exact same 69 JSON contract scenarios as #47 (byte-identical fixture file). Additional tests exercise real PyGithub pagination and mutation methods through a stub transport, Enterprise URLs, output serialization, validation, and error handling. YAML lint, action/workflow schemas, jactionlint, and diff checks passed. Dedicated CI tests Linux AMD64 and ARM64 and runs the actual composite against this PR with read-only credentials and dry_run enabled.
Dependency selection
Latest head validation: Linux AMD64 and ARM64 tests plus the actual read-only composite smoke test both passed. CI run.
🚨 To Reviewers