-
Notifications
You must be signed in to change notification settings - Fork 624
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
Support for the Data Service Specification #738
base: master
Are you sure you want to change the base?
Conversation
@laeubi can you check the failing CI? |
@gotson as it complains about code formatting is ther an Eclipse format profile or something I can use to format the source code? |
You can use the Spotless plugin: |
1f95d55
to
5ac503f
Compare
Done! |
This has no unit tests, is this feasible to add some ? |
I'll try to add test-cases here, but have first to check how sqlite tests are structured. |
I've checked the issue you raised at osgi/osfgi-test and i think you framed it quite well: how to make it easier for maintainers that know nothing and have no particular interest in OSGI to gauge a PR and have confidence to merge it. That probably will take a bit of time, but i think it's the right approach. |
@gotson it hase taken a bit longer than desired but now I finally was able to add a small test-case, do you like to review and give some feedback if that would suffice? |
The more i think of it, the more i wonder whether this needs to be part of |
As always there are many options but in general this has several drawbacks, as already described here:
Beside that an extra project always needs to be found, needs extra release management and so on, also having "official" support let people directly contribute to the project instead of potentially contribute to two projects. |
Add a first basic implementation of the DataSourceFactory
It seems I missed formatting the test code, fixed now ... |
Add a first basic implementation of the DataSourceFactory