Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@stianst
Copy link
Contributor

@stianst stianst commented Oct 4, 2023

This introduces a maven property, project.version.npm, to allow maven to know what NPM version is, in scenarios that it's not identical to project.version. This occurs when the set-version's semver translation code is activated

Closes #23635

This introduces a maven property, `project.version.npm`, to allow maven to know what NPM version is, in scenarios that it's not identical to `project.version`. This occurs when the set-version's semver translation code is activated

Closes keycloak#23635
@stianst stianst requested review from a team as code owners October 4, 2023 06:04
@ghost ghost added team/ui labels Oct 4, 2023
@cypress
Copy link

cypress bot commented Oct 4, 2023

1 flaky test on run #9207 ↗︎

0 527 48 0 Flakiness 1

Details:

Merge d2f1753 into bfc1238...
Project: Keycloak Admin UI Commit: ea1c82c62a ℹ️
Status: Passed Duration: 16:02 💡
Started: Oct 4, 2023 6:17 AM Ended: Oct 4, 2023 6:33 AM
Flakiness  cypress/e2e/authentication_test.spec.ts • 1 flaky test • chrome

View Output Video

Test Artifacts
Authentication test > should add a condition Output Screenshots

Review all test suite changes for PR #23680 ↗︎

@cypress
Copy link

cypress bot commented Oct 4, 2023

1 flaky test on run #9208 ↗︎

0 527 48 0 Flakiness 1

Details:

Fix set-version.sh's handling of NPM versions (#23638) (#23680)
Project: Keycloak Admin UI Commit: d64bfc50c6
Status: Passed Duration: 17:50 💡
Started: Oct 4, 2023 7:33 AM Ended: Oct 4, 2023 7:51 AM
Flakiness  cypress/e2e/authentication_test.spec.ts • 1 flaky test • chrome

View Output Video

Test Artifacts
Authentication test > should add a condition Output Screenshots

Review all test suite changes for PR #23680 ↗︎

@stianst stianst mentioned this pull request Nov 14, 2023
@stianst stianst deleted the backport-22.0-23628 branch July 17, 2024 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants