merge service loader implementation by mkrebs81 · Pull Request #28 · SmartGridready/SGrJavaDrivers · GitHub
Skip to content

merge service loader implementation - #28

Merged
mkrebs81 merged 12 commits into
masterfrom
26-use-serviceloader
Nov 27, 2024
Merged

merge service loader implementation#28
mkrebs81 merged 12 commits into
masterfrom
26-use-serviceloader

Conversation

@mkrebs81

Copy link
Copy Markdown
Contributor

No description provided.

@mkrebs81 mkrebs81 linked an issue Nov 18, 2024 that may be closed by this pull request
@mkrebs81
mkrebs81 requested a review from ergo-furrer November 25, 2024 07:40
@mkrebs81
mkrebs81 marked this pull request as ready for review November 25, 2024 07:41
@mkrebs81

Copy link
Copy Markdown
Contributor Author

@ergo-furrer

ergo-furrer commented Nov 26, 2024

Copy link
Copy Markdown
Contributor

@ergo-furrer the current state should work with the latest driver API. I am not fully sure about the versioning of the MQTT and REST drivers... should I keep it at 1.0.0 or set it to 2.0.0 directly?

@mkrebs81 I would be more comfortable with 2.0.0, so it is more aligned with the Commhandler version.

ergo-furrer
ergo-furrer previously approved these changes Nov 26, 2024
@mkrebs81

Copy link
Copy Markdown
Contributor Author

@mkrebs81
mkrebs81 merged commit 7fae8c0 into master Nov 27, 2024
@mkrebs81
mkrebs81 deleted the 26-use-serviceloader branch November 28, 2024 08:29
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.

Use ServiceLoader for driver implementations

2 participants