-
-
Notifications
You must be signed in to change notification settings - Fork 9
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Code remastering to fix warning "Detected I/O inside the event loop" #2
- Loading branch information
Showing
21 changed files
with
1,135 additions
and
404 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,79 @@ | ||
<!-- | ||
You are amazing! Thanks for contributing to our project! | ||
Please, DO NOT DELETE ANY TEXT from this template! (unless instructed). | ||
--> | ||
## Breaking change | ||
<!-- | ||
If your PR contains a breaking change for existing users, it is important | ||
to tell them what breaks, how to make it work again and why we did this. | ||
This piece of text is published with the release notes, so it helps if you | ||
write it towards our users, not us. | ||
Note: Remove this section if this PR is NOT a breaking change. | ||
--> | ||
|
||
|
||
## Proposed change | ||
<!-- | ||
Describe the big picture of your changes here to communicate to the | ||
maintainers why we should accept this pull request. If it fixes a bug | ||
or resolves a feature request, be sure to link to that issue in the | ||
additional information section. | ||
--> | ||
|
||
|
||
## Type of change | ||
<!-- | ||
What type of change does your PR introduce to Home Assistant? | ||
NOTE: Please, check only 1! box! | ||
If your PR requires multiple boxes to be checked, you'll most likely need to | ||
split it into multiple PRs. This makes things easier and faster to code review. | ||
--> | ||
|
||
- [ ] Dependency upgrade | ||
- [ ] Bugfix (non-breaking change which fixes an issue) | ||
- [ ] New integration (thank you!) | ||
- [ ] New feature (which adds functionality to an existing integration) | ||
- [ ] Breaking change (fix/feature causing existing functionality to break) | ||
- [ ] Code quality improvements to existing code or addition of tests | ||
|
||
## Example entry for `configuration.yaml`: | ||
<!-- | ||
Supplying a configuration snippet, makes it easier for a maintainer to test | ||
your PR. Furthermore, for new integrations, it gives an impression of how | ||
the configuration would look like. | ||
Note: Remove this section if this PR does not have an example entry. | ||
--> | ||
|
||
```yaml | ||
# Example configuration.yaml | ||
|
||
``` | ||
|
||
## Additional information | ||
<!-- | ||
Details are important, and help maintainers processing your PR. | ||
Please be sure to fill out additional details, if applicable. | ||
--> | ||
|
||
- This PR fixes or closes issue: fixes # | ||
- This PR is related to issue: | ||
|
||
## Checklist | ||
<!-- | ||
Put an `x` in the boxes that apply. You can also fill these out after | ||
creating the PR. If you're unsure about any of them, don't hesitate to ask. | ||
We're here to help! This is simply a reminder of what we are going to look | ||
for before merging your code. | ||
--> | ||
|
||
- [ ] The code change is tested and works locally. | ||
- [ ] There is no commented out code in this PR. | ||
- [ ] The code has been formatted using Black (`black --fast homeassistant tests`) | ||
|
||
If user exposed functionality or configuration variables are added/changed: | ||
|
||
- [ ] Documentation added/updated to README.md | ||
|
||
<!-- | ||
Thank you for contributing <3 | ||
--> |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,18 +1,58 @@ | ||
repos: | ||
- repo: https://github.com/asottile/pyupgrade | ||
rev: v2.4.1 | ||
hooks: | ||
- id: pyupgrade | ||
args: [--py37-plus] | ||
- repo: https://github.com/psf/black | ||
rev: 19.10b0 | ||
hooks: | ||
- id: black | ||
args: | ||
- --safe | ||
- --quiet | ||
files: ^(custom_components|script)/.+\.py$ | ||
- repo: https://gitlab.com/pycqa/flake8 | ||
rev: 3.8.1 | ||
hooks: | ||
- id: flake8 | ||
additional_dependencies: | ||
- flake8-docstrings==1.5.0 | ||
- pydocstyle==5.0.2 | ||
files: ^(custom_components)/.+\.py$ | ||
- repo: https://github.com/pre-commit/pre-commit-hooks | ||
rev: v2.5.0 | ||
hooks: | ||
- id: check-executables-have-shebangs | ||
stages: [manual] | ||
- id: check-json | ||
- id: trailing-whitespace | ||
# - id: no-commit-to-branch | ||
# args: | ||
# - --branch=master | ||
- repo: https://github.com/adrienverge/yamllint.git | ||
rev: v1.23.0 | ||
hooks: | ||
- id: yamllint | ||
- repo: local | ||
hooks: | ||
# Run mypy through our wrapper script in order to get the possible | ||
# pyenv and/or virtualenv activated; it may not have been e.g. if | ||
# committing from a GUI tool that was not launched from an activated | ||
# shell. | ||
- id: mypy | ||
name: mypy | ||
entry: script/run-in-env.sh mypy | ||
language: script | ||
types: [python] | ||
require_serial: true | ||
files: ^custom_components/.+\.py$ | ||
- id: update-tracker | ||
name: "Update Tracker" | ||
entry: ./update_tracker.py | ||
entry: script/update_tracker.py | ||
language: system | ||
- id: pylint | ||
name: pylint | ||
entry: python3 -m pylint.__main__ -d R0801 | ||
entry: python3 -m pylint.__main__ | ||
language: system | ||
types: [python] | ||
- repo: https://github.com/pre-commit/pre-commit-hooks | ||
rev: v2.5.0 | ||
hooks: | ||
- id: check-json | ||
- id: check-yaml | ||
- id: trailing-whitespace |
Oops, something went wrong.