# Code Review Template ``` ## Meta * Platform: * Mycroft-core version: * Who: [@your-github-handle](https://github.com/your-github-handle) / @your-mycroft-chat-handle * Datestamp: run `date +%Y-%m-%d_%H:%M:%S_%Z` * Language and dialect of tester: ## 0. Automated tests Are all automated tests passing? * [ ] Skill tester - Jenkins * [ ] Continuous Integration - Travis-CI ## 1. Code Review - secure and stable * [ ] **Code Quality** Can you understand what the code is doing? Is there inline documentation? Do you have any concerns about this code running on your machine? Are there any performance issues such as nested or infinite loops? Do you have significant concerns about the overall code quality? _NOTE: We do not enforce PEP8 Checks on Skills_ > * [ ] **Error Handling** Are there any specific checks we make for error handling or graceful degradation? > * [ ] **Libraries** Does the Skill include the correct libraries? Does it use too many libraries or dependencies? > * [ ] **Required Dependencies** Check `requirements.txt` and `requirements.sh` - are the required dependencies listed? If `requirements.sh` is used, is some form of conditional processing done to match against multiple distros? Often Skill Authors will add requirements.txt using only an “library=1.x.x” instead of “library >=1.x.x”. Check to make sure that there is an equal or greater than in the requirements to help future-proof the Skill, unless a _specific_ version is needed. > * [ ] **Settings** Is the `settingsmeta` file well laid out? If settings are not used, has the default file been deleted? If it is the default file, the first setting section will be called "Options << Name of section". > * [ ] **Integration Tests** Does the skill include sufficient integration tests, included in the `test` folder? > * [ ] **Other Files** Are there any other files included that are unnecessary or you are unsure of their function? > ### Actions Required: \* ``` --- # Agent Instructions: Querying This Documentation If you need additional information that is not directly available in this page, you can query the documentation dynamically by asking a question. Perform an HTTP GET request on the current page URL with the `ask` query parameter: ``` GET https://mycroft-ai.gitbook.io/docs/skill-development/marketplace-submission/skills-acceptance-process/skills-review-code-template.md?ask= ``` The question should be specific, self-contained, and written in natural language. The response will contain a direct answer to the question and relevant excerpts and sources from the documentation. Use this mechanism when the answer is not explicitly present in the current page, you need clarification or additional context, or you want to retrieve related documentation sections.