Skip to content

feat: auto-pin SHA + testable validation scripts#2

Merged
davidtranhp merged 2 commits into
masterfrom
feat/auto-pin
May 12, 2026
Merged

feat: auto-pin SHA + testable validation scripts#2
davidtranhp merged 2 commits into
masterfrom
feat/auto-pin

Conversation

@davidtranhp
Copy link
Copy Markdown
Contributor

Summary

  • Refactor inline Python from validate.yml into scripts/validate_schema.py and scripts/validate_reachability.py (callable as CLI tools)
  • Add tests/ with 16 unit tests covering schema validation and reachability checks (403, 404, auth header injection)
  • Add .github/workflows/test.yml to run pytest on scripts/+tests/ changes
  • Update CONTRIBUTING.md: document SHA-based versioning (no version field in plugin.json), add auto-pin setup guide with pin-sha.yml template, label/PAT/branch-protection checklist
  • Update README.md to reference auto-pinning

Required manual setup (after merge)

  • Create label auto-pin in this repo
  • Enable Allow auto-merge in repo settings
  • Add branch protection on master: require status check validate

Test plan

  • pytest tests/ -v → 16 passed
  • Trigger validate.yml manually → both steps pass using scripts

Comment thread .github/workflows/test.yml Fixed
- Refactor inline Python from validate.yml into scripts/validate_schema.py
  and scripts/validate_reachability.py (both callable as CLI tools)
- Add tests/ with 16 unit tests covering schema validation and
  reachability checks (403, 404, auth header injection)
- Add .github/workflows/test.yml to run pytest on scripts/tests/ changes
- Update CONTRIBUTING.md: document SHA-based versioning (no version field
  in plugin.json), add auto-pin setup guide with pin-sha.yml template,
  label/PAT/branch-protection checklist
- Update README.md to reference auto-pinning and CONTRIBUTING guide
@davidtranhp davidtranhp merged commit 17e1439 into master May 12, 2026
4 checks passed
@davidtranhp davidtranhp deleted the feat/auto-pin branch May 12, 2026 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants