docs(examples): add Hermes Tweet skill - #17
Conversation
📝 WalkthroughWalkthroughThis PR adds a Hermes Tweet skill card covering metadata, workflows, permissions, safety, examples, testing, and release requirements, and updates the quick-test script to propagate Rust test failures. ChangesHermes Tweet Skill Documentation
Quick Test Failure Handling
Estimated code review effort: 2 (Simple) | ~10 minutes 🚥 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 |
|
Published independent repository fix in 4387d6e: scripts/test_quick.sh now propagates Rust test failures instead of swallowing them with an unconditional success override. Validation passed with 95 Rust unit tests and 7 integration tests. I also re-audited the full conversation, reviews, inline threads, checks, commits, and base divergence; no unresolved maintainer request or conflict remains. |
Summary
examples/skills.scripts/test_quick.shso Rust test failures fail the quick-test command instead of being silently ignored.Independent Repository Fix
The quick-test script runs with
set -euo pipefail, but its Rust pipeline ended with|| true. That override converted every Rust test failure into success. Commit4387d6eremoves the override so the repository's primary quick check now reports real Rust failures.Validation
bash -n scripts/test_quick.shcargo test --lib --test skill_session_tests: 95 unit tests and 7 integration tests passed.git diff --check upstream/mainSummary by CodeRabbit
skill_session_testsbreak.