-
Notifications
You must be signed in to change notification settings - Fork 87
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
Backport main
fixes/dependency updates for 2.2.1
#1241
Conversation
Bumps [swagger-ui-dist](https://github.com/swagger-api/swagger-ui) from 4.13.2 to 4.14.0. - [Release notes](https://github.com/swagger-api/swagger-ui/releases) - [Commits](swagger-api/swagger-ui@v4.13.2...v4.14.0) --- updated-dependencies: - dependency-name: org.webjars.npm:swagger-ui-dist dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
Fixes smallrye#1218 Signed-off-by: Michael Edgar <michael@xlate.io>
- Do not include properties where the name is not a valid Java identifier (covers the case of generated methods for Kotlin value class fields) - Add test data module, initally holding test Kotlin classes Fixes smallrye#1195 Signed-off-by: Michael Edgar <michael@xlate.io>
Signed-off-by: Michael Edgar <michael@xlate.io>
Signed-off-by: Michael Edgar <michael@xlate.io>
Signed-off-by: Michael Edgar <michael@xlate.io>
Signed-off-by: Michael Edgar <michael@xlate.io>
Previously an ApplicationArchiveProcessor was used to run the implementation and generate the OpenAPI document in the archive before deploying it. However, and ApplicationArchiveProcessor is only called for _testable_ archives (archives where tests run on the server), whereas to pass the TCK we need to process all the archives. Instead, use the BeforeDeploy event to update the archive which is called for every deployment.
Bumps `version.quarkus` from 2.11.2.Final to 2.12.0.Final. Updates `resteasy-reactive-common` from 2.11.2.Final to 2.12.0.Final Updates `quarkus-kotlin` from 2.11.2.Final to 2.12.0.Final Updates `quarkus-reactive-routes` from 2.11.2.Final to 2.12.0.Final --- updated-dependencies: - dependency-name: io.quarkus.resteasy.reactive:resteasy-reactive-common dependency-type: direct:development update-type: version-update:semver-minor - dependency-name: io.quarkus:quarkus-kotlin dependency-type: direct:development update-type: version-update:semver-minor - dependency-name: io.quarkus:quarkus-reactive-routes dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
Fixes smallrye#1202 Signed-off-by: Michael Edgar <michael@xlate.io>
Bumps smallrye-parent from 35 to 36. --- updated-dependencies: - dependency-name: io.smallrye:smallrye-jakarta-parent dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Michael Edgar <michael@xlate.io>
Signed-off-by: Michael Edgar <michael@xlate.io>
Kudos, SonarCloud Quality Gate passed! |
@radcortez - we need this merged and 2.2.1 released before updating 3.0.x for the Jakarta migration release. |
@@ -0,0 +1,131 @@ | |||
name: "SonarCloud" |
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.
Just curious about why you split the Sonar step? We may want to do it as well for other SR components.
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.
When it runs for a PR, this repo's Sonar secrets are not available to push the scan results.
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.
This allows it to work for both PRs and local repo pushes, although arguably PRs are more useful to have the scan.
to-jakarta.sh
script only)node
directory #1222