Skip to content
This repository was archived by the owner on Nov 6, 2025. It is now read-only.

Added objects. Validate letters. Add distributions#10

Closed
peternijssen wants to merge 1 commit intoiMicknl:masterfrom
peternijssen:master
Closed

Added objects. Validate letters. Add distributions#10
peternijssen wants to merge 1 commit intoiMicknl:masterfrom
peternijssen:master

Conversation

@peternijssen
Copy link
Copy Markdown
Contributor

My enviroment is acting strange. Will have to check what's wrong, but perhaps meanwhile you can validate code. Thanks!

@peternijssen
Copy link
Copy Markdown
Contributor Author

@iMicknl Any updates on code validation? I will run some final tests on my end.

@iMicknl
Copy link
Copy Markdown
Owner

iMicknl commented Aug 4, 2019

Sorry, I totally missed this PR... Will have a look today / tomorrow! Thanks for your contribution.

@peternijssen
Copy link
Copy Markdown
Contributor Author

No problem! I still have to look why it won't run locally. I believe it's a config error in my editor though. So I don't expect many more code changes.

@peternijssen
Copy link
Copy Markdown
Contributor Author

Closing this one in favor of #11

@iMicknl
Copy link
Copy Markdown
Owner

iMicknl commented Aug 8, 2019

@peternijssen I will have a look at all changes this week, so we can get it published and create a new Home Assistant PR.

iMicknl pushed a commit that referenced this pull request Aug 18, 2019
* Added objects. Validate letters. Add distributions
* small fixes in package and letter, added relevant method and black formatted
* refined classes, added to changelog and updated test_nl...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants