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

[modbus] Replace apache commons lang usage with plain java #10002

Merged
merged 3 commits into from
Feb 1, 2021

Conversation

ssalonen
Copy link
Contributor

@ssalonen ssalonen commented Jan 31, 2021

Related: #7722

After this PR, only Transformation class in modbus uses apache commons lang. I will remove the final references in #9945: 36b45ed

Signed-off-by: Sami Salonen ssalonen@gmail.com

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
ssalonen added a commit to ssalonen/openhab2-addons that referenced this pull request Jan 31, 2021
- see also openhab#10002
- I removed equals and hashCode implementation all-together, I could not
  see they played any role in practice.

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
Signed-off-by: Sami Salonen <ssalonen@gmail.com>
Signed-off-by: Sami Salonen <ssalonen@gmail.com>
Copy link
Member

@Hilbrand Hilbrand left a comment

Choose a reason for hiding this comment

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

LGTM

@Hilbrand Hilbrand merged commit d5e83e3 into openhab:main Feb 1, 2021
@Hilbrand Hilbrand added this to the 3.1 milestone Feb 1, 2021
ssalonen added a commit to ssalonen/openhab2-addons that referenced this pull request Feb 1, 2021
- see also openhab#10002
- I removed equals and hashCode implementation all-together, I could not
  see they played any role in practice.

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
Hilbrand pushed a commit that referenced this pull request Feb 2, 2021
… new-style transformation string (#9945)

* [modbus] Cascaded transforms with ∩
* [modbus] README to mention cascaded transformations
* [modbus] Take cascaded transformation into use
* [modbus] README to show preference towards new syntax
* [modbus] examples to use new syntax
* [modbus] fix test
* [modbus] remove apache commons lang dependency
- see also PR #10002
- I removed equals and hashCode implementation all-together, I could not see they played any role in practice.

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
lucacalcaterra pushed a commit to lucacalcaterra/openhab-addons that referenced this pull request Feb 23, 2021
…0002)

* [modbus] fix item creation in tests

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
Signed-off-by: Luca Calcaterra <calcaterra.luca@gmail.com>
lucacalcaterra pushed a commit to lucacalcaterra/openhab-addons that referenced this pull request Feb 23, 2021
… new-style transformation string (openhab#9945)

* [modbus] Cascaded transforms with ∩
* [modbus] README to mention cascaded transformations
* [modbus] Take cascaded transformation into use
* [modbus] README to show preference towards new syntax
* [modbus] examples to use new syntax
* [modbus] fix test
* [modbus] remove apache commons lang dependency
- see also PR openhab#10002
- I removed equals and hashCode implementation all-together, I could not see they played any role in practice.

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
Signed-off-by: Luca Calcaterra <calcaterra.luca@gmail.com>
themillhousegroup pushed a commit to themillhousegroup/openhab2-addons that referenced this pull request May 10, 2021
…0002)

* [modbus] fix item creation in tests

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
Signed-off-by: John Marshall <john.marshall.au@gmail.com>
themillhousegroup pushed a commit to themillhousegroup/openhab2-addons that referenced this pull request May 10, 2021
… new-style transformation string (openhab#9945)

* [modbus] Cascaded transforms with ∩
* [modbus] README to mention cascaded transformations
* [modbus] Take cascaded transformation into use
* [modbus] README to show preference towards new syntax
* [modbus] examples to use new syntax
* [modbus] fix test
* [modbus] remove apache commons lang dependency
- see also PR openhab#10002
- I removed equals and hashCode implementation all-together, I could not see they played any role in practice.

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
Signed-off-by: John Marshall <john.marshall.au@gmail.com>
splatch pushed a commit to ConnectorIO/openhab-addons that referenced this pull request Aug 19, 2021
… new-style transformation string (openhab#9945)

* [modbus] Cascaded transforms with ∩
* [modbus] README to mention cascaded transformations
* [modbus] Take cascaded transformation into use
* [modbus] README to show preference towards new syntax
* [modbus] examples to use new syntax
* [modbus] fix test
* [modbus] remove apache commons lang dependency
- see also PR openhab#10002
- I removed equals and hashCode implementation all-together, I could not see they played any role in practice.

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
thinkingstone pushed a commit to thinkingstone/openhab-addons that referenced this pull request Nov 7, 2021
…0002)

* [modbus] fix item creation in tests

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
thinkingstone pushed a commit to thinkingstone/openhab-addons that referenced this pull request Nov 7, 2021
… new-style transformation string (openhab#9945)

* [modbus] Cascaded transforms with ∩
* [modbus] README to mention cascaded transformations
* [modbus] Take cascaded transformation into use
* [modbus] README to show preference towards new syntax
* [modbus] examples to use new syntax
* [modbus] fix test
* [modbus] remove apache commons lang dependency
- see also PR openhab#10002
- I removed equals and hashCode implementation all-together, I could not see they played any role in practice.

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
splatch pushed a commit to ConnectorIO/openhab-addons that referenced this pull request Apr 23, 2022
… new-style transformation string (openhab#9945)

* [modbus] Cascaded transforms with ∩
* [modbus] README to mention cascaded transformations
* [modbus] Take cascaded transformation into use
* [modbus] README to show preference towards new syntax
* [modbus] examples to use new syntax
* [modbus] fix test
* [modbus] remove apache commons lang dependency
- see also PR openhab#10002
- I removed equals and hashCode implementation all-together, I could not see they played any role in practice.

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
…0002)

* [modbus] fix item creation in tests

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
… new-style transformation string (openhab#9945)

* [modbus] Cascaded transforms with ∩
* [modbus] README to mention cascaded transformations
* [modbus] Take cascaded transformation into use
* [modbus] README to show preference towards new syntax
* [modbus] examples to use new syntax
* [modbus] fix test
* [modbus] remove apache commons lang dependency
- see also PR openhab#10002
- I removed equals and hashCode implementation all-together, I could not see they played any role in practice.

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
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