Skip to content
This repository has been archived by the owner on Sep 8, 2024. It is now read-only.

Cleanup of core.py #1835

Merged
merged 5 commits into from
Oct 10, 2018
Merged

Conversation

forslund
Copy link
Collaborator

@forslund forslund commented Oct 7, 2018

Description

  • Fix remaining bare except
  • Replace inner function mutable with nonlocal in get_scheduled_event_status()
  • create helper function simple_trace()
  • Improve readability of get_handler_name()

How to test

Check that mycroft works normally

Contributor license agreement signed?

CLA [ Yes ]

The new simple_trace() function can now generate a simplified stack trace instead of doing it inline in the MycroftSkill class
The most notable things:
- Use the new wait flag when calling speak
- Fix except without defined exception
@forslund forslund added Type: Refactoring and other improvements Improvement of code and documentation that does not alter functionality. CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) labels Oct 7, 2018
@pep8speaks
Copy link

Hello @forslund! Thanks for submitting the PR.

Copy link
Contributor

@LearnedVector LearnedVector left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@LearnedVector LearnedVector merged commit 6289bb5 into MycroftAI:dev Oct 10, 2018
@forslund forslund deleted the refactor/skills-core branch February 18, 2019 09:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) Type: Refactoring and other improvements Improvement of code and documentation that does not alter functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants