Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Resolvers: Allow ranges / npm tags for version resolution #649

Merged
merged 9 commits into from
Sep 5, 2023

Conversation

RandomByte
Copy link
Member

@RandomByte RandomByte commented Aug 24, 2023

The framework version resolution now supports all valid node semver
ranges and uses tags from the corresponding registry for version
resolution (not supported for Sapui5MavenSnapshotResolver).

JIRA: CPOUI5FOUNDATION-722

@RandomByte RandomByte requested a review from matz3 August 24, 2023 14:36
@matz3 matz3 changed the title [FEATURE] Resolvers: Use npm tags for determining 'latest' [FEATURE] Resolvers: Allow ranges / npm tags for version resolution Aug 30, 2023
@coveralls
Copy link

coveralls commented Aug 30, 2023

Coverage Status

coverage: 95.706% (-0.2%) from 95.881% when pulling 4de6bff on resolver-use-npm-tags into b62f860 on main.

@matz3 matz3 marked this pull request as ready for review September 1, 2023 13:00
@matz3 matz3 requested a review from a team September 1, 2023 13:02
@flovogt flovogt force-pushed the resolver-use-npm-tags branch 2 times, most recently from 336eddc to ce4243e Compare September 4, 2023 12:59
@matz3 matz3 added the enhancement New feature or request label Sep 4, 2023
matz3 added a commit to SAP/ui5-cli that referenced this pull request Sep 5, 2023
Updates according to new features via SAP/ui5-project#649

JIRA: CPOUI5FOUNDATION-722
RandomByte and others added 9 commits September 5, 2023 11:24
The framework version resolution now supports all valid node semver
ranges and uses tags from the corresponding registry for version
resolution (not supported for Sapui5MavenSnapshotResolver).
- Adds tests to clarify that only -SNAPSHOT ranges should use
the "includePrerelease" behavior.
- Remove duplicate tests
- Remove obsolete SEMVER regular expression
There is no real need to have it in sync with the configuration schema.
If semver would change something, we would anyways not notice it via
that test as both expressions are defined by us.
matz3 added a commit to SAP/ui5-cli that referenced this pull request Sep 5, 2023
Updates according to new features via SAP/ui5-project#649

JIRA: CPOUI5FOUNDATION-722
@matz3 matz3 merged commit 8b667ef into main Sep 5, 2023
17 checks passed
@matz3 matz3 deleted the resolver-use-npm-tags branch September 5, 2023 11:21
matz3 added a commit to SAP/ui5-cli that referenced this pull request Sep 5, 2023
Updates according to new features via SAP/ui5-project#649

JIRA: CPOUI5FOUNDATION-722
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants