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

Javax -> Jakarta #1374

Merged
merged 3 commits into from
May 2, 2022
Merged

Javax -> Jakarta #1374

merged 3 commits into from
May 2, 2022

Conversation

phillip-kruger
Copy link
Member

This PR will for the base for the 2.0 release (jakarta namespace)

Signed-off-by: Phillip Kruger <[email protected]>
Signed-off-by: Phillip Kruger <[email protected]>
@phillip-kruger phillip-kruger added this to the 2.0.0 milestone Apr 30, 2022
@phillip-kruger phillip-kruger requested a review from jmartisk April 30, 2022 09:15
@phillip-kruger
Copy link
Member Author

@jmartisk - There are some test failing in integration tests module. I'll have a look again a.s.a.p, but you can have a look so long.

@phillip-kruger
Copy link
Member Author

@jmartisk I disabled some tests so that CI can pass. I will then do a RC1 release so that we have a jakarta release out. Before a final release we need to solve the issues. They are all in integration-tests :

Errors: 
  DynamicClientInjectionTest.testInjectedClient » IllegalArgument ArquillianServ...
  DynamicClientInjectionTest.verifyHttpHeaders » IllegalArgument ArquillianServl...
  MetricTest.verifyMetricsAreRegisteredEagerly:49 NullPointer
  MetricTest.verifyMetricsAreUpdated:77 NullPointer
  ComplexOpenTracingTest.prepare:48 NullPointer

Can you have a look and see if you no why we have these issues ? I tried upgrading to Junit5 as well, but I still get the same errors.

Let me know

@phillip-kruger phillip-kruger marked this pull request as ready for review May 2, 2022 05:26
@phillip-kruger phillip-kruger merged commit d274e03 into smallrye:main May 2, 2022
@phillip-kruger
Copy link
Member Author

@jmartisk - Merged with failures as mentioned. Doing a RC1 release now. From now on 1.5.x will contain that javax version and main (this) the jakarta. Any new features and bug fixes should go into both

@phillip-kruger phillip-kruger deleted the jakarta branch May 2, 2022 05:37
Comment on lines +99 to +106
<artifactItem>
<groupId>org.wildfly</groupId>
<artifactId>wildfly-preview-dist</artifactId>
<version>${wildfly.version}</version>
<type>zip</type>
<overWrite>false</overWrite>
<outputDirectory>${project.build.directory}</outputDirectory>
</artifactItem>
Copy link
Member Author

Choose a reason for hiding this comment

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

@jmartisk ^^^

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.

1 participant