Skip to content

chore: remove Python from the repository tooling#5

Merged
mergify[bot] merged 1 commit into
mainfrom
devs/jd/worktree-remove-python/remove-python-repo-tooling--82858ba0
Jul 1, 2026
Merged

chore: remove Python from the repository tooling#5
mergify[bot] merged 1 commit into
mainfrom
devs/jd/worktree-remove-python/remove-python-repo-tooling--82858ba0

Conversation

@jd

@jd jd commented Jun 29, 2026

Copy link
Copy Markdown
Member

The action's runtime is pure shell, but its dev/CI tooling still pulled in
Python: a uv-run generate-doc.py for the README inputs table, and a
pip-installed semgrep + yamllint in CI. Remove it all so the repo matches its
own "No Python or toolchain is required" promise.

  • generate-doc.py is rewritten as pure bash + awk in generate-doc.sh: it parses
    action.yml and rewrites the inputs table between the AUTO-DOC-INPUT markers.
    Output is byte-identical to the previous generator (verified on both BSD awk
    and Ubuntu's mawk), so the autodoc drift check still passes. awk emits the
    finished Markdown rows directly, avoiding a TSV/read round-trip whose
    IFS-whitespace tab handling would have collapsed an empty Default column.
  • CI linters: drop actions/setup-python and pip install semgrep yamllint; run
    yamllint and semgrep from their pinned docker images, matching the existing
    actionlint step. semgrep sets an explicit entrypoint so a digest bump can't
    break the invocation. Renovate manages the tags + digests.
  • autodoc job: drop the uv setup step; it now runs the pure-bash script.

No Python, uv, or pip remains in the repo.

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com

The action's runtime is pure shell, but its dev/CI tooling still pulled in
Python: a uv-run generate-doc.py for the README inputs table, and a
pip-installed semgrep + yamllint in CI. Remove it all so the repo matches its
own "No Python or toolchain is required" promise.

- generate-doc.py is rewritten as pure bash + awk in generate-doc.sh: it parses
  action.yml and rewrites the inputs table between the AUTO-DOC-INPUT markers.
  Output is byte-identical to the previous generator (verified on both BSD awk
  and Ubuntu's mawk), so the autodoc drift check still passes. awk emits the
  finished Markdown rows directly, avoiding a TSV/`read` round-trip whose
  IFS-whitespace tab handling would have collapsed an empty Default column.
- CI linters: drop actions/setup-python and `pip install semgrep yamllint`; run
  yamllint and semgrep from their pinned docker images, matching the existing
  actionlint step. semgrep sets an explicit entrypoint so a digest bump can't
  break the invocation. Renovate manages the tags + digests.
- autodoc job: drop the uv setup step; it now runs the pure-bash script.

No Python, uv, or pip remains in the repo.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Change-Id: I82858ba068c90064336fe309a9c8996a9899dea8
@mergify mergify Bot requested a review from a team June 29, 2026 17:09
@mergify

mergify Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 All 6 merge protections satisfied — ready to merge.

Show 6 satisfied protections

🟢 🤖 Continuous Integration

  • all of:
    • check-success=all-greens

🟢 👀 Review Requirements

  • any of:
    • #approved-reviews-by>=2
    • author = dependabot[bot]
    • author = mergify-ci-bot
    • author = renovate[bot]

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?:

🟢 🔎 Reviews

  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 PR description

  • body ~= (?ms:.{48,})

🟢 🚦 Auto-queue

When all merge protections are satisfied, this pull request will be queued automatically.

@mergify mergify Bot deployed to Mergify Merge Protections June 29, 2026 17:09 Active
@jd jd closed this Jun 29, 2026
@jd jd reopened this Jun 30, 2026
@jd jd marked this pull request as ready for review July 1, 2026 09:36
@jd jd requested a review from a team July 1, 2026 09:36
@mergify

mergify Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • Entered queue2026-07-01 13:16 UTC · Rule: default · triggered by merge protections
  • Checks skipped · PR is already up-to-date
  • Merged2026-07-01 13:16 UTC · at 6bfa01a13783665702e8fadee11ef219b3b96dab · squash

This pull request spent 28 seconds in the queue, including 3 seconds running CI.

Required conditions to merge

@mergify mergify Bot merged commit 075911f into main Jul 1, 2026
16 checks passed
@mergify mergify Bot added queued and removed queued labels Jul 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants