justinmclean commented on PR #213: URL: https://github.com/apache/airflow-steward/pull/213#issuecomment-4527385131
Thanks, this is all addressed now. I rebased `setup-security` onto latest `main` and dropped the duplicate merge/conflict-cleanup noise, so the branch is no longer carrying the inflated merge history. I also fixed the two validator gaps you called out: - `_code_spans` now handles single-backtick inline spans that wrap newlines, with a regression test for the `--body\n"<x>"` case. - The placeholder check now also catches `-F <text-field>=<placeholder>` without `@file`, while avoiding false positives for scalar GraphQL vars like `owner`, `repo`, `number`, and node IDs. The cosmetic `security-issue-triage/SKILL.md` line-join workaround is gone from the final diff. Local verification: - `uv run --project tools/skill-validator --group dev pytest tools/skill-validator/tests/test_validator.py` - `uv run --project tools/skill-validator skill-validate` - `uv run --project tools/skill-validator skill-validate --strict` - `prek run --from-ref upstream/main` All pass locally. I’ll force-push the rebased branch and re-request review. -- 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]
