[RFC] 00010 UI5 Builder-Bundling Refactoring#583
Merged
Conversation
2 tasks
…code at the end of a bundle
Similar to Array.filter
b1ed587 to
6c6f066
Compare
Member
Author
|
|
RandomByte
added a commit
to SAP/ui5-builder
that referenced
this pull request
Feb 23, 2022
Resolves UI5/cli#472 Supersedes #282 Based on UI5/cli#583 JIRA: CPOUI5FOUNDATION-434 Co-authored-by: Matthias Osswald <mat.osswald@sap.com>
codeworrior
reviewed
Mar 22, 2022
|
|
||
| ### resourceRoot-mappings | ||
|
|
||
| When bundling modules via resourceRoot-mappings, it might become impossible to reference the original source from a bundle's source map. For example, for a bundle located in `/test-resources`, which includes modules from `/resources`, the source map must not reference the original module via `../../resources/x/y/z` as this path rather depends on the resourceRoot-mapping and might be different at runtime. |
Member
There was a problem hiding this comment.
LGTM.
Line 172 IMO should be moved into the "Source Map support" section, too.
flovogt
reviewed
Mar 22, 2022
|
|
||
| The outdated annotation convention `//@` should not be supported. All known relevant tools (Terser, TypeScript compiler) are using the new convention. See [Conventions](http://sourcemaps.info/spec.html#h.lmz475t4mvbx) in the source map specification for details. | ||
|
|
||
| Any source maps found for a resource shall be added to the source map of the bundle. Either use an [index map](http://sourcemaps.info/spec.html#h.535es3xeprgt), containing the individual source maps, or generate a new source map for the whole bundle based off them. |
Member
There was a problem hiding this comment.
It's "based of them", isn't it?
|
|
||
|
|
||
| <!-- | ||
| This is the bulk of the RFC. Explain the design in enough detail for somebody familiar with the UI5 Tooling to understand, and for somebody familiar with the implementation to implement. This should get into specifics and corner-cases, and include examples of how the feature is used. Any new terminology should be defined here. |
Member
There was a problem hiding this comment.
There also further comments below
|
|
||
| ## Drawbacks | ||
|
|
||
| To be discussed. |
Member
There was a problem hiding this comment.
if we have none, could we write it here.
|
|
||
| ## Alternatives | ||
|
|
||
| To be discussed. |
Member
There was a problem hiding this comment.
if we have none, could we write it here.
d3xter666
pushed a commit
that referenced
this pull request
Sep 24, 2025
Resolves #472 Supersedes SAP/ui5-builder#282 Based on #583 JIRA: CPOUI5FOUNDATION-434 Co-authored-by: Matthias Osswald <mat.osswald@sap.com>
d3xter666
pushed a commit
that referenced
this pull request
Sep 24, 2025
…1.2.4 (#583) * Bump coverallsapp/github-action from 1.2.0 to 1.2.4 Bumps [coverallsapp/github-action](https://github.com/coverallsapp/github-action) from 1.2.0 to 1.2.4. - [Release notes](https://github.com/coverallsapp/github-action/releases) - [Commits](coverallsapp/github-action@v1.2.0...v1.2.4) --- updated-dependencies: - dependency-name: coverallsapp/github-action dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * [INTERNAL] Remove manual configured github token With 1.2.4, defaulting automatically to github token. --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Florian Vogt <florian.vogt@sap.com>
d3xter666
pushed a commit
that referenced
this pull request
Sep 25, 2025
) Resolves #472 Supersedes SAP/ui5-builder#282 Based on #583 JIRA: CPOUI5FOUNDATION-434 Co-authored-by: Matthias Osswald <mat.osswald@sap.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Read it here: RFC 0010 UI5 Builder-Bundling Refactoring
PoC: SAP/ui5-builder#657