Skip to content

[KEYCLOAK-19862] Integrate the quarkus-ecosystem-ci#8971

Closed
andreaTP wants to merge 1 commit intokeycloak:mainfrom
andreaTP:quarkus-ecosystem-ci
Closed

[KEYCLOAK-19862] Integrate the quarkus-ecosystem-ci#8971
andreaTP wants to merge 1 commit intokeycloak:mainfrom
andreaTP:quarkus-ecosystem-ci

Conversation

@andreaTP
Copy link
Contributor

This PR along with this:
quarkusio/quarkus-ecosystem-ci#69

Should cover the technical implementation of:
https://github.com/quarkusio/quarkus-ecosystem-ci#what-do-i-need-to-do-as-a-quarkus-platform-participant

We need someone with higher privileges than me over this organization to go through the steps.

Note that I do expect one of #8963 or #8964 to be merged before this one.

@andreaTP andreaTP requested a review from pedroigor November 30, 2021 15:10
mkdir -p ~/.m2 ; cp .github/settings.xml ~/.m2/

# update Quarkus dependencies
# TODO: aligning parent pom versions with upstream Quarkus breaks compilation
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@pedroigor / @DGuhr this is something we probably want to target, running a command like:

mvn versions:compare-dependencies \
    -pl :keycloak-parent \
    -DremotePom=io.quarkus:quarkus-bom:${QUARKUS_VERSION} \
    -DupdatePropertyVersions=true \
    -DupdateDependencies=true \
    -DgenerateBackupPoms=false

breaks the compilation.

Copy link
Contributor

@stianst stianst left a comment

Choose a reason for hiding this comment

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

This needs discussing first

@abstractj abstractj added the status/needs-discussion PR needs discussion on developer mailing list label Mar 15, 2022
@abstractj
Copy link
Contributor

@andreaTP I believe we can take a step back and close it. Lets proceed with the discussions. Considering that has been a while since our last interaction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/dist/quarkus status/needs-discussion PR needs discussion on developer mailing list

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants