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

Skip to content

Conversation

@ASzc
Copy link
Contributor

@ASzc ASzc commented Oct 2, 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

@ASzc ASzc requested review from a team as code owners October 2, 2023 15:33
@ghost ghost added team/ui labels Oct 2, 2023
@cypress
Copy link

cypress bot commented Oct 2, 2023

1 flaky test on run #9196 ↗︎

0 527 48 0 Flakiness 1

Details:

Merge 1e69ba9 into e6e724d...
Project: Keycloak Admin UI Commit: f918ff29da ℹ️
Status: Passed Duration: 13:57 💡
Started: Oct 3, 2023 2:24 PM Ended: Oct 3, 2023 2:38 PM
Flakiness  cypress/e2e/realm_settings_general_tab_test.spec.ts • 1 flaky test • chrome

View Output Video

Test Artifacts
Realm settings general tab tests > Test all general tab switches Test Replay Output Screenshots

Review all test suite changes for PR #23638 ↗︎

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 merged commit 7953085 into keycloak:main Oct 4, 2023
stianst pushed a commit to stianst/keycloak that referenced this pull request 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 keycloak#23635
stianst added a commit that referenced this pull request 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

Co-authored-by: Alex Szczuczko <[email protected]>
@stianst stianst mentioned this pull request Nov 14, 2023
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.

JS adapter build fails if project.version is updated to a version requiring semver translation

3 participants