-
Notifications
You must be signed in to change notification settings - Fork 813
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
JakartaEE modules #810
JakartaEE modules #810
Conversation
c25e0e6
to
41f4485
Compare
41f4485
to
26bb35e
Compare
FYI @jdyer1 |
I agree this is the right thing to do. We shouldn't break dev experience with their IDEs! |
pom.xml
Outdated
@@ -305,10 +303,10 @@ | |||
<useDefaultExcludes>true</useDefaultExcludes> | |||
<addDefaultLicenseMatchers>false</addDefaultLicenseMatchers> | |||
<licenses> | |||
<licens implementation="org.apache.rat.analysis.license.ApacheSoftwareLicense20" /> | |||
<licens implementation="org.apache.rat.analysis.license.ApacheSoftwareLicense20"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a typo?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, it's a typo but it doesn't matter as this an element of an array. Yet I fixed it :)
</plugins> | ||
</build> | ||
<dependencies> | ||
<dependency> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jdyer1 Do you recall why this dependency was required? It doesn't seem like we needed it before
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like a valid dependency
[ERROR] import org.apache.velocity.tools.ToolContext;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh I see, it was previously included transitively by velocity-tools-view
, makes sense
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
Moves JakartaEE related modules into a dedicated subdirectory
Depends on #809