Skip to content

Introduce Pico ConfigBean Builder Extensions #5482

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

Merged
merged 37 commits into from
Dec 2, 2022

Conversation

trentjeff
Copy link
Member

For #5443 .

Note that there are some todo items left like validation. It definitely needs a draft review before I go much further though.

@trentjeff trentjeff added 4.x Version 4.x declarative Helidon Declarative labels Nov 21, 2022
@trentjeff trentjeff added this to the 4.0.0 milestone Nov 21, 2022
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Nov 21, 2022
@trentjeff trentjeff changed the title Introduce pico ConfigBean Builder extensions Introduce Pico ConfigBean Builder Extensions Nov 21, 2022
trentjeff and others added 22 commits November 21, 2022 17:40
…en extend ConfigBeanMapper to be a specialization of ConfigMapperProvider
* remove -Ddocker.build=true

Signed-off-by: aserkes <[email protected]>
tempDir support for tests

Signed-off-by: aserkes <[email protected]>
* Upgrade GraalVM native image to 22.3.0 (helidon-io#5308)

* Introducing native image plugin from GraalVM
* Support GraalVM 22.3 build
* Update graalvm version in all relevant files.

Signed-off-by: Tomas Langer <[email protected]>

* Upgrade GraalVM native image to 22.3.0

Signed-off-by: Tomas Langer <[email protected]>
* Restored Tyrus module in Nima MP.

Signed-off-by: Santiago Pericasgeertsen <[email protected]>

* Updated dependencies and socket routing code.

Signed-off-by: Santiago Pericasgeertsen <[email protected]>

* Initial integration of Tyrus with Nima. Simple echo test works using the JDK's WS client.

* Fixed checkstyle issues.

Signed-off-by: Santiago Pericasgeertsen <[email protected]>

* Simplified TyrusConnection by not extending WsConnection. Restored some changes in Nima's WS module.

* Some general code cleanup.

Signed-off-by: Santiago Pericasgeertsen <[email protected]>

* Renamed some classes for consistency. Refactored tests for declarative and programmatic styles and to use @HelidonTest.

* Support for @RoutingPath annotation.

* Fixed checkstyle errors.

* New test for extensions that uses the Tyrus client after fixes in Nima.

* New test that combines REST and WS endpoint on same port.

* Restored examples that use websockets.

Signed-off-by: Santiago Pericasgeertsen <[email protected]>

* Order of protected fields and use putIfAbsent.

Signed-off-by: Santiago Pericasgeertsen <[email protected]>

Signed-off-by: Santiago Pericasgeertsen <[email protected]>
…#5536)

* Fixed OpentraceableClientE2ETest to be more deterministic (and added explanation of what it should test)
Disabled BulkheadTest that is failing intermittently

* Fixed tests that leave current span scope open (and influence other tests)
* Fix test for multiple try and errors.

* Fix imports.
* remove -Ddocker.build=true

Signed-off-by: aserkes <[email protected]>

* fix copyrights

Signed-off-by: aserkes <[email protected]>

* Create a custom exception to remove OutOfMemoryError in CharBuffer

Signed-off-by: aserkes <[email protected]>

* remove unnecessary constructors

Signed-off-by: aserkes <[email protected]>

Signed-off-by: aserkes <[email protected]>
@trentjeff trentjeff marked this pull request as ready for review November 30, 2022 01:46
@trentjeff trentjeff mentioned this pull request Dec 2, 2022
@trentjeff trentjeff merged commit 4c90786 into helidon-io:main Dec 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.x Version 4.x declarative Helidon Declarative OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

10 participants