fix: Fix for Codex parsing of skill.#948
Conversation
Try out this PRQuick install: pip install --upgrade --force-reinstall git+https://github.com/ai-dynamo/aiperf.git@22666a45ffc2793b39a3db8e735a93f4d1c1479fRecommended with virtual environment (using uv): uv venv --python 3.12 && source .venv/bin/activate
uv pip install --upgrade --force-reinstall git+https://github.com/ai-dynamo/aiperf.git@22666a45ffc2793b39a3db8e735a93f4d1c1479fLast updated for commit: |
Signed-off-by: Francesco Di Natale <3429989+FrankD412@users.noreply.github.com>
48bb1f8 to
d2921a9
Compare
|
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 selected for processing (1)
WalkthroughThe PR rewrites a single sentence in the skill definition document to correct the grammar of the "Explicitly out of scope" clause, changing from colon-based phrasing to "are" form without altering the listed items or any code behavior. ChangesSkill Description Grammar Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
This PR simply changes a single word to fix a colon error when loading the the ergonomics skill. The colon in the description gets flagged as a YAML parsing issue as below:
Summary by CodeRabbit