-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[shelly] Support for Duo, EM3, DW, Smoke, Addon; new CoAP-based updates; bug fixes #6985
Conversation
Travis tests were successfulHey @markus7017, |
Travis tests were successfulHey @markus7017, |
2 similar comments
Travis tests were successfulHey @markus7017, |
Travis tests were successfulHey @markus7017, |
Travis tests were successfulHey @markus7017, |
4 similar comments
Travis tests were successfulHey @markus7017, |
Travis tests were successfulHey @markus7017, |
Travis tests were successfulHey @markus7017, |
Travis tests were successfulHey @markus7017, |
Travis tests have failedHey @markus7017, 2nd BuildExpand here
|
Travis tests have failedHey @markus7017, 2nd BuildExpand here
|
Travis tests were successfulHey @markus7017, |
Travis tests have failedHey @markus7017, 1st BuildExpand here
|
Travis tests were successfulHey @markus7017, |
Travis tests were successfulHey @markus7017, |
1 similar comment
Travis tests were successfulHey @markus7017, |
Signed-off-by: Markus Michels <markus7017@gmail.com>
@cpmeister added synchronized to start() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is more work that could be done but this is good enough for now.
Just need to make sure that the build is fine then I can merge. |
Signed-off-by: Markus Michels <markus7017@gmail.com>
Travis tests were successfulHey @markus7017, |
@cpmeister Well done! I appreciate your help and patience. Thank you. Do you have advises what can be improved ot should be refactored? |
One note: The last PR was not mentioned in the release notes. I checked with Kai and he mentioned that maybe tagging the PR as enhancement and as bug might caused this issue. Could you check that? I have a breaking change (channel layout for the RGBW2). Where should that be documented? |
As per J-N-K:
|
Which PR was that? |
It should not be there at all - patch releases are supposed to be fully backward compatible! |
FTR: I'd suggest not to merge such huge PRs the day before a release - we cannot be sure that it is fully finished (see discussion above) and that it has been thoroughly enough tested by people (see #7416). As we do monthly releases atm, it should always be safer to keep such changes for the next release. |
@kaikreuzer I might have gotten carried away and merged this too soon. @markus7017 Please make sure that your future PRs are single purpose. Do not group together different bug fixes and enhancements into a single PR; it can turn a PR into an ever-growing set of changes that become a nightmare to review. The harder something is to review the less likely it will be reviewed/merged in a timely manner. |
…es; bug fixes (openhab#6985) * Re-checkin based on latest PR review status Signed-off-by: Markus Michels <markus7017@gmail.com> Signed-off-by: Eugen Freiter <freiter@gmx.de>
…es; bug fixes (openhab#6985) * Re-checkin based on latest PR review status Signed-off-by: Markus Michels <markus7017@gmail.com>
…es; bug fixes (openhab#6985) * Re-checkin based on latest PR review status Signed-off-by: Markus Michels <markus7017@gmail.com>
…es; bug fixes (openhab#6985) * Re-checkin based on latest PR review status Signed-off-by: Markus Michels <markus7017@gmail.com>
…es; bug fixes (openhab#6985) * Re-checkin based on latest PR review status Signed-off-by: Markus Michels <markus7017@gmail.com> Signed-off-by: CSchlipp <christian@schlipp.de>
…es; bug fixes (openhab#6985) * Re-checkin based on latest PR review status Signed-off-by: Markus Michels <markus7017@gmail.com>
…es; bug fixes (openhab#6985) * Re-checkin based on latest PR review status Signed-off-by: Markus Michels <markus7017@gmail.com>
…es; bug fixes (openhab#6985) * Re-checkin based on latest PR review status Signed-off-by: Markus Michels <markus7017@gmail.com>
…es; bug fixes (openhab#6985) * Re-checkin based on latest PR review status Signed-off-by: Markus Michels <markus7017@gmail.com>
…es; bug fixes (openhab#6985) * Re-checkin based on latest PR review status Signed-off-by: Markus Michels <markus7017@gmail.com> Signed-off-by: Daan Meijer <daan@studioseptember.nl>
…es; bug fixes (openhab#6985) * Re-checkin based on latest PR review status Signed-off-by: Markus Michels <markus7017@gmail.com>
Includes Fixes and enhancements based on PR #6764
Signed-off-by: Markus Michels markus7017@gmail.com