Skip to content
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

[energidataservice] Refactor to eliminate code duplication #15651

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

jlaur
Copy link
Contributor

@jlaur jlaur commented Sep 25, 2023

While working on #15636, I was reminded about code duplication that increased the risk of bugs due to the potential mixing of variables, such as through copy-and-paste errors.

@jlaur jlaur force-pushed the energidataservice-refactoring branch from afde04b to 4f9a590 Compare September 25, 2023 20:23
@jlaur jlaur changed the title [energidataservice] Refactor to avoid code duplication [energidataservice] Refactor to eliminate code duplication Sep 25, 2023
@jlaur jlaur force-pushed the energidataservice-refactoring branch 2 times, most recently from 6c96360 to d691657 Compare September 25, 2023 20:36
@jlaur jlaur marked this pull request as ready for review September 25, 2023 20:38
Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
@jlaur jlaur force-pushed the energidataservice-refactoring branch from d691657 to 4219347 Compare September 30, 2023 19:55
@jlaur jlaur requested a review from a team September 30, 2023 19:56
Copy link
Member

@wborn wborn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks much better indeed!

@wborn wborn merged commit 86f8bc9 into openhab:main Oct 2, 2023
2 checks passed
@wborn wborn added this to the 4.1 milestone Oct 2, 2023
@jlaur jlaur deleted the energidataservice-refactoring branch October 2, 2023 15:36
pat-git023 pushed a commit to pat-git023/openhab-addons that referenced this pull request Oct 13, 2023
…15651)

Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
querdenker2k pushed a commit to querdenker2k/openhab-addons that referenced this pull request Oct 21, 2023
…15651)

Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
querdenker2k pushed a commit to querdenker2k/openhab-addons that referenced this pull request Oct 29, 2023
…15651)

Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
Signed-off-by: querdenker2k <querdenker2k@gmx.de>
austvik pushed a commit to austvik/openhab-addons that referenced this pull request Mar 27, 2024
…15651)

Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants