-
-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: intent api indentation error #653
Conversation
remove deprecated sigint handler from mycroft times
WalkthroughThis pull request removes a call to a signal reset function from the main execution path and adjusts the control flow in the intent handling logic. The signal handler reset import and its invocation are deleted, indicating a shift in SIGINT management. Additionally, the return statement in an intent-processing method is re-indented to ensure it executes only when a condition is met. No new functionality is introduced. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant IntentService
Client->>IntentService: handle_get_intent(intent_data)
loop Iterate over potential intents
IntentService->>IntentService: Evaluate intent match condition
alt Condition met
IntentService->>Client: Emit intent reply and return value
else Condition not met
IntentService->>IntentService: Continue evaluating next intent
end
end
Suggested labels
Poem
✨ Finishing Touches
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
ovos_core/__main__.py
(0 hunks)ovos_core/intent_services/__init__.py
(1 hunks)
💤 Files with no reviewable changes (1)
- ovos_core/main.py
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: license_tests
- GitHub Check: end2end_tests (3.9)
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #653 +/- ##
==========================================
- Coverage 75.33% 71.10% -4.24%
==========================================
Files 15 15
Lines 3094 1651 -1443
==========================================
- Hits 2331 1174 -1157
+ Misses 763 477 -286
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
* Translate intents.json via GitLocalize (#651) Co-authored-by: timonvanhasselt <timonvanhasselt@visio.org> * Increment Version to 0.5.3a1 * Update Changelog * fix: intent api indentation error (#653) remove deprecated sigint handler from mycroft times * Increment Version to 0.5.4a1 * Update Changelog --------- Co-authored-by: gitlocalize-app[bot] <55277160+gitlocalize-app[bot]@users.noreply.github.com> Co-authored-by: timonvanhasselt <timonvanhasselt@visio.org> Co-authored-by: JarbasAl <JarbasAl@users.noreply.github.com> Co-authored-by: JarbasAI <33701864+JarbasAl@users.noreply.github.com>
remove deprecated sigint handler from mycroft times
Summary by CodeRabbit
Bug Fixes
Refactor