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] generateFlexChangesBundle: Handle minUI5Version as array #1009

Merged
merged 1 commit into from
Apr 24, 2024

Conversation

Lonwyr
Copy link
Contributor

@Lonwyr Lonwyr commented Apr 9, 2024

generateFlexChangesBundle understands minUI5Version as string and array

With the upcoming UI5 2.x the manifest may contain a minUI5Version with multiple entries as described in the AppDescriptor schema (internal PR 499). This change enables the generateFlexChangesBundle to understand the new syntax

in addition, the tests for the generateFlexChangesBundle were enhanced to check better their complete functionality - boy scouted!

JIRA: CLOUDIOI-1170

Pull Request Checklist

Copy link

cla-assistant bot commented Apr 9, 2024

CLA assistant check
All committers have signed the CLA.

@Lonwyr Lonwyr force-pushed the minUI5VersionArrays branch from ea5d00c to df3c9bb Compare April 9, 2024 12:47
@coveralls
Copy link

coveralls commented Apr 16, 2024

Coverage Status

coverage: 94.613% (+0.005%) from 94.608%
when pulling 0c64ca4 on Lonwyr:minUI5VersionArrays
into 1b57ce0 on SAP:v3.

@flovogt flovogt requested a review from a team April 17, 2024 13:05
@flovogt
Copy link
Member

flovogt commented Apr 17, 2024

LGTM. I would prefer to change the base branch of this PR to "v3". So we can release this feature together with a UI5 Tooling v3 minor release.

@d3xter666
Copy link
Contributor

d3xter666 commented Apr 18, 2024

LGTM

LGTM. I would prefer to change the base branch of this PR to "v3". So we can release this feature together with a UI5 Tooling v3 minor release.

We had a quick discussion today. For v4 (main branch) it should be INTERNAL and for v3 it should be a FEATURE.
It shouldn't matter where it was merged first as long as the commit message is clear and correct

@flovogt flovogt changed the base branch from main to v3 April 18, 2024 09:57
@Lonwyr Lonwyr force-pushed the minUI5VersionArrays branch from 1be174e to 1004610 Compare April 18, 2024 10:12
@Lonwyr
Copy link
Contributor Author

Lonwyr commented Apr 18, 2024

The pull request was update to target v3.

flovogt
flovogt previously approved these changes Apr 18, 2024
@flovogt flovogt self-requested a review April 18, 2024 10:16
@Lonwyr Lonwyr force-pushed the minUI5VersionArrays branch 2 times, most recently from b4c1f21 to f5da889 Compare April 18, 2024 10:30
…ng and array

With the upcoming UI5 2.x the manifest may contain a minUI5Version with multiple entries. This change enables the generateFlexChangesBundle to understand the new syntax.
The language check kindly pointed out some words to improve. With this commit the proposals of the check are applied.
@Lonwyr Lonwyr force-pushed the minUI5VersionArrays branch from f5da889 to 0c64ca4 Compare April 19, 2024 09:04
@RandomByte RandomByte changed the title [FEATURE] generateFlexChangesBundle understands minUI5Version as stri… [FEATURE] generateFlexChangesBundle: Handle minUI5Version as array Apr 24, 2024
@RandomByte RandomByte merged commit 45c9b7e into SAP:v3 Apr 24, 2024
14 checks passed
RandomByte pushed a commit that referenced this pull request Apr 24, 2024
…1009)

With the upcoming UI5 2.x the manifest may contain a minUI5Version with
multiple entries as described in the AppDescriptor schema (internal PR
499). This change enables the generateFlexChangesBundle to understand
the new syntax

in addition, the tests for the generateFlexChangesBundle were enhanced
to check better their complete functionality - boy scouted!

JIRA: CLOUDIOI-1170
(cherry picked from commit 45c9b7e)
@RandomByte
Copy link
Member

Cherry-pick for main: #1011

RandomByte pushed a commit that referenced this pull request Apr 24, 2024
…1009)

With the upcoming UI5 2.x the manifest may contain a minUI5Version with
multiple entries as described in the AppDescriptor schema (internal PR
499). This change enables the generateFlexChangesBundle to understand
the new syntax

in addition, the tests for the generateFlexChangesBundle were enhanced
to check better their complete functionality - boy scouted!

JIRA: CLOUDIOI-1170
(cherry picked from commit 45c9b7e)
@Lonwyr Lonwyr deleted the minUI5VersionArrays branch April 25, 2024 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants