Skip to content

Fix set-version.sh's handling of NPM versions (#23638) - #23680

Merged
stianst merged 1 commit into
keycloak:release/22.0from
stianst:backport-22.0-23628
Oct 4, 2023
Merged

Fix set-version.sh's handling of NPM versions (#23638)#23680
stianst merged 1 commit into
keycloak:release/22.0from
stianst:backport-22.0-23628

Conversation

@stianst

@stianst stianst commented Oct 4, 2023

Copy link
Copy Markdown
Contributor

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

cypress Bot commented Oct 4, 2023

Copy link
Copy Markdown

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

cypress Bot commented Oct 4, 2023

Copy link
Copy Markdown

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