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] Bundling: Remove option 'usePredefineCalls' #725

Closed
wants to merge 1 commit into from

Conversation

flovogt
Copy link
Member

@flovogt flovogt commented May 21, 2024

Up until UI5 Tooling v3, the bundle option "usePredefineCalls" defaults to "false" and has to be activated explicitly in a custom bundle configuration.
For default bundles, such as Component-preload or self-contained bundle (sap-ui-custom.js), there is no way to use the option apart from re-defining the whole bundle via custom bundle definition.

With UI5 Tooling v4 bundles are generated with the usage sap.ui.predefine calls instead of the former default function wrapper. This leads to smaller bundle sizes and less overhead at runtime.

As the option only affects the internal handling of bundling without affecting the actual users,
the option is removed completely, instead of just changing the default value of the option.

Related to SAP/ui5-builder#1021.

Documentation update covered in SAP/ui5-tooling#957.

JIRA: CPOUI5FOUNDATION-760

@coveralls
Copy link

Coverage Status

coverage: 95.762%. remained the same
when pulling f55e3d2 on remove-usePredefineCalls-option
into 8b6b26d on main.

@flovogt flovogt requested a review from a team May 21, 2024 13:25
@@ -397,10 +397,6 @@
"type": "boolean",
"default": false
},
"usePredefineCalls": {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This changes existing specVersions, which is not intended

@flovogt flovogt added the blocked Pull requests that are blocked label May 22, 2024
@flovogt
Copy link
Member Author

flovogt commented May 24, 2024

The update of the schema will be handled by a new specVersion 4.0 (JIRA: CPOUI5FOUNDATION-835). Therefore closing this PR.

@flovogt flovogt closed this May 24, 2024
@flovogt flovogt deleted the remove-usePredefineCalls-option branch May 24, 2024 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Pull requests that are blocked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants