Conversation
since moving to pipeline plugins the plugin_id is expected as key https://github.com/OpenVoiceOS/ovos-plugin-manager/blob/dev/ovos_plugin_manager/pipeline.py#L91
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughConfiguration keys under the intents pipeline were renamed to reflect plugin-specific naming conventions. Four configuration keys—"adapt", "padatious", "common_query", and "OCP"—were updated to their plugin-specific equivalents while preserving all nested configuration values. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
* fix: mycroft.conf key related PR OpenVoiceOS/ovos-config#264 * 📝 Add docstrings to `JarbasAl-patch-2` Docstrings generation was requested by @JarbasAl. The following files were modified: * `ovos_commonqa/opm.py` * Update opm.py --------- Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #264 +/- ##
=====================================
Coverage 0.00% 0.00%
=====================================
Files 10 10
Lines 837 857 +20
=====================================
- Misses 837 857 +20 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
since moving to pipeline plugins the plugin_id is expected as key
https://github.com/OpenVoiceOS/ovos-plugin-manager/blob/dev/ovos_plugin_manager/pipeline.py#L91
Summary by CodeRabbit