Subham-KRLX commented on PR #64012:
URL: https://github.com/apache/airflow/pull/64012#issuecomment-4130922117
Found and fixed a hook compatibility issue While testing locally, I
discovered that Airflow's existing insert-license pre-commit hook automatically
places an SPDX HTML comment at the top of every SKILL.md file This conflicted
with the strict content.startswith("---\n") check in validate_skills.py, making
the validate-agent-skills-standard hook always fail.
Fixed in the latest commit by updating validate_skills.py to skip a leading
HTML comment block before checking for YAML frontmatter — keeping both hooks
fully compatible. All 298 local tests still pass Happy to hear if you'd prefer
a different approach (e.g. excluding SKILL.md from insert-license via
.pre-commit-config.yaml).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
