Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/_checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ jobs:
persist-credentials: false
- uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 # v5.6.0
with:
python-version: "3.12"
python-version: "3.14"

@devin-ai-integration devin-ai-integration Bot Aug 6, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

πŸ” Version pin is the only Python reference; inlined script must stay in sync with wave-foundation

The bump at .github/workflows/_checks.yml:59 is the only Python version reference in the repo (no other workflow, pyproject.toml, or .py sources pin a version), so there is no in-repo inconsistency introduced. However, the comment at .github/workflows/_checks.yml:48-50 states this job "mirrors wave-foundation/scripts/validate-skills.py; keep the two in sync" β€” the upstream copy of this workflow may still pin 3.12, so the two will drift unless updated together. The inlined script only uses re, yaml, subprocess, and os, all of which are unaffected by 3.14 behavior changes, and pip install pyyaml on 3.14 requires PyYAML wheels for cp314 (available in recent releases); worth confirming the gate job actually runs green on the PR before merge.

Open in Devin Review

Was this helpful? React with πŸ‘ or πŸ‘Ž to provide feedback.

- run: pip install pyyaml
- name: Skill frontmatter gate
run: |
Expand Down
Loading