-
-
Notifications
You must be signed in to change notification settings - Fork 26
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
Updates Based on Integration Blueprint #171
Updates Based on Integration Blueprint #171
Conversation
I created a separate branch for these updates so the fixes can be done there without impacting the main branch (could also solve #169 at the same time as well). I also added in the ruff formatter as well to the action which will automatically format and it will commit the changes automatically as well though since the lint action fails it doesn't commit anything at the moment. Can fix that by using the --exit-zero argument so the auto commit step gets reached. I also tested the --fix argument on a separate branch if you want to see how it works. EDIT: Also noticed that there's still a reference to the old API signup page and it should be updated to use APIable:
|
Cleaned up more of the lint errors and now there's only 4 things left which I might leave for you to fix. I also fixed the deprecated constants issue in sensor.py as well. I haven't tested to see if everything is still running without issues (I don't have a dev machine to test and I don't have Visual Studio code installed either) but it validating is a good sign? EDIT: I tried to update a personal branch with the latest changes and it wouldn't let me fix the merge commits on GitHub without updating master for some reason. Anyway I reverted the changes here. I also fixed the last remaining errors but still unsure that nothing will totally break. |
3cb7ed9
into
Pirate-Weather:blueprint-update
I've noticed that there's been updates to the integration blueprint since the creation of this repo and this PR adds in the items that are not contained in this repo.
Ruff has been added to the blueprint as a workflow which is a Python linter/code formatter and after getting it to run on my repo it showed 359 errors.