Skip to content

[KEYCLOAK-19862] Nightly quarkus tests based on published Snapshots#8963

Closed
andreaTP wants to merge 1 commit intokeycloak:mainfrom
andreaTP:test-published-nightlies
Closed

[KEYCLOAK-19862] Nightly quarkus tests based on published Snapshots#8963
andreaTP wants to merge 1 commit intokeycloak:mainfrom
andreaTP:test-published-nightlies

Conversation

@andreaTP
Copy link
Contributor

@andreaTP andreaTP commented Nov 30, 2021

This nightly job test Keycloak against the published snapshot versions of Quarkus.

Discussion

@andreaTP andreaTP force-pushed the test-published-nightlies branch from 5e3efd1 to a35cb87 Compare November 30, 2021 11:12
@andreaTP
Copy link
Contributor Author

Alternative to #8964

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.

I'd like to discuss this before we move ahead with this one.

@abstractj
Copy link
Contributor

@stianst the PR needs a rebase. Would you like @andreaTP to create a GH discussion about this?

@abstractj abstractj added the status/needs-discussion PR needs discussion on developer mailing list label Feb 23, 2022
@andreaTP
Copy link
Contributor Author

@abstractj a discussion already exists:
#8985

@abstractj abstractj removed the status/needs-discussion PR needs discussion on developer mailing list label Feb 23, 2022
@abstractj abstractj requested a review from stianst February 23, 2022 18:35
@abstractj abstractj assigned DGuhr and pedroigor and unassigned DGuhr Feb 23, 2022
@stianst stianst added the status/hold PR should not be merged. On hold for later. label Feb 25, 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.

@andreaTP andreaTP closed this Apr 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/dist/quarkus status/hold PR should not be merged. On hold for later.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants