fix(gws): resolve skillsaw linting warnings#63
Conversation
Fix weak language ("Note that", "gracefully"), broken internal links
(missing https://developers.google.com prefix), and scope the
gws/skills/** exclude to only the context-budget rule.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR restructures the ChangesConfig and Version Updates
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Poem A rabbit hops through yaml lines, 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@stbenjam these skills are generated from the |
|
There's broken links in them was mostly that I wanted to fix but it's not a big deal. While developing skillsaw I usually run the new release through a bunch of repos with the skillsaw.yaml deleted so I can see the impact of any changes including new rules |
|
fixing the broken links in #68 |
Summary
https://developers.google.comprefixgws/skills/**exclude in.skillsaw.yamlto only thecontext-budgetrule instead of globally excluding all rulesTest plan
uvx skillsaw --strictpasses with 0 warnings and 0 errors🤖 Generated with Claude Code
Summary by CodeRabbit