-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
jakarta-web relax the import package constraints #3791
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
Conversation
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.
LGTM
hmm what is exactly the problem? should i add some spaces somewhere or remove some? |
It is a formatting problem, run: ./mvnw spotless:apply to fix it and push the changes. |
Head branch was pushed to by a user without write access
7368f0d
to
329a0f2
Compare
329a0f2
to
3522618
Compare
A snapshot should be ready as soon as the workflow finishes: https://github.com/apache/logging-log4j2/actions/runs/15978983571 |
type="changed"> | ||
<issue id="3787" link="https://github.com/apache/logging-log4j2/issues/3787"/> | ||
<description format="asciidoc"> | ||
add a bnd tool property that makes sure that the supported jakarta version has a wider range then 5 to 6 |
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.
than
instead of then
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.
Thanks, I'll keep it mind, while reviewing the changelog entries for the release.
fixes: #3787
Checklist
Before we can review and merge your changes, please go through the checklist below. If you're still working on some items, feel free to submit your pull request as a draft—our CI will help guide you through the remaining steps.
✅ Required checks
[ X] License: I confirm that my changes are submitted under the Apache License, Version 2.0.
[ X] Commit signatures: All commits are signed and verifiable. (See GitHub Docs on Commit Signature Verification).
[ X] Code formatting: The code is formatted according to the project’s style guide.
How to check and fix formatting
./mvnw spotless:check
./mvnw spotless:apply
See the build instructions for details.
Build & Test: I verified that the project builds and all unit tests pass.
How to build the project
Run:
./mvnw verify
See the build instructions for details.
🧪 Tests (select one)
📝 Changelog (select one)
src/changelog/.2.x.x
. (See Changelog Entry File Guide).