Repository navigation
Retire package-version bump and synchronization automation - #136
Eduardo Villalpando Mello (edvilme) merged 3 commits into
Conversation
Co-authored-by: edvilme <5952839+edvilme@users.noreply.github.com>
|
🔒 Automated review in progress — Rich Chiodo (@rchiodo) is auto-reviewing this PR. |
There was a problem hiding this comment.
🟡 Changes recommended
CONTRIBUTING.md incorrectly presents same-revision source builds as universal, despite Black still using a packaged TypeScript runtime and requiring a separate B0 migration.
1 open issue
What changed in this PR
Retires package-version bookkeeping in favor of pinned Git revisions as the compatibility identity, while leaving package metadata and Python diagnostic version reporting unchanged.
Changes:
- Removes the weekly patch-bump workflow and both version-consistency checks.
- Deletes the root
VERSIONfile and version-management helpers. - Documents commit-based adoption, optional milestone releases, and deferred S4 artifact-provenance enforcement.
| File | Description |
|---|---|
VERSION |
Removes the shared version source. |
scripts/version/validate.py |
Removes manifest-version validation. |
scripts/version/sync.py |
Removes manifest-version synchronization. |
scripts/version/bump_patch.py |
Removes patch-bump automation. |
scripts/version/__init__.py |
Removes shared version-management utilities. |
CONTRIBUTING.md |
Replaces release bookkeeping instructions with source-adoption guidance. |
.github/workflows/version-check.yml |
Removes the version-consistency check. |
.github/workflows/pr-version.yml |
Removes the duplicate version-match check. |
.github/workflows/auto-patch-bump.yml |
Removes scheduled and manual patch-bump automation. |
Review effort: Balanced
|
Copilot resolve the merge conflicts in this pull request |
…e-version-bump # Conflicts: # CONTRIBUTING.md Co-authored-by: edvilme <5952839+edvilme@users.noreply.github.com>
Merged |
|
Reviewed commit: Result: 🔴 Verification detailsVerification: Automated verification is best effort. The PR author remains responsible for testing. Summary: [unavailable] Container verification could not start and local execution was not authorized for this PR HEAD: Podman machine 'pyrx-automation' is not running. Test runs: none recorded. |
Rich Chiodo (rchiodo)
left a comment
There was a problem hiding this comment.
Approved via Review Center.

Consumers identify compatibility by pinned Git revision, making synchronized npm/Python version bookkeeping unnecessary. Package metadata remains fixed for local build, installation, and diagnostics.
Automation cleanup
scripts/version/and rootVERSION.Metadata behavior
Documentation
Consumer and repository coordination
^0.8.2alongside the Python submodule; its source migration remains separate.