docs(en): merge rollup/master into rollup-docs-cn/master @ 8faa1877#213
Open
docschina-bot wants to merge 26 commits into
Open
docs(en): merge rollup/master into rollup-docs-cn/master @ 8faa1877#213docschina-bot wants to merge 26 commits into
docschina-bot wants to merge 26 commits into
Conversation
* Improve documentation for manualPureFunctions * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <[email protected]> --------- Co-authored-by: Copilot Autofix powered by AI <[email protected]>
…ing PTR lookups (#6404) * fix: set report.excludeNetwork=true before getReport() to avoid blocking PTR lookups * test: skip excludeNetwork test on Windows where getReport() runs in a child process --------- Co-authored-by: Lukas Taegert-Atkinson <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* chore(deps): update dependency concurrently to v10 * Pin previous concurrently We need to update our NodeJS baseline first. --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Lukas Taegert-Atkinson <[email protected]>
…te chunks (#6374) * Add a test * Extract the static dependencies imported by manual chunks into separate chunks * Adjust the log info * Update the code according to Copilot’s review * fix: use some instead of find for boolean check * test: add test for shared static dependency between manual chunks * Avoid unnecessary array Initially, we are only collection single modules with their dependent entries. * Avoid full iteration by gathering entry information * Treat manual chunks as entry points for chunk assignment This will effectively avoid cycles where manual chunks import back from entry chunks. The price, though, is that it might create more chunks. For the longer term, it would be nice to replace the current logic where non-explicit manual chunks try to include as many static dependencies as possible with a logic where they behave as any entry points and just include all dependencies not shared with another entry. * Reduce number of chunks by treating all manual chunk members as dependencies Basically for the purpose of analysis, all explicit members of a manual chunk are treated as the same module, meaning dependencies are shared in the chunk. This mirrors what will happen in the output and reduces generated chunks. * Refactor manual chunk assembly * Track if we already added a manual chunk * Update codecov action as it is failing --------- Co-authored-by: Lukas Taegert-Atkinson <[email protected]>
* chore(deps): lock file maintenance minor/patch updates * Resolve audit --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Lukas Taegert-Atkinson <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* chore(deps): lock file maintenance minor/patch updates * Resolve audit --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Lukas Taegert-Atkinson <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* fix(deps): update minor/patch updates * Fix dependency update --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Lukas Taegert-Atkinson <[email protected]>
…Filter (#6415)
The wildcard pattern matching in testFilter checked whether the value
started with the prefix (parts[0]) but did not advance the search
position past it before checking the suffix. This caused false positive
matches when the prefix and suffix were the same string and the value was
exactly equal to one copy of that string.
For example, the pattern 'code:foo*foo' incorrectly matched { code: 'foo' }
because 'foo'.startsWith('foo') is true AND 'foo'.endsWith('foo') is true,
even though the wildcard '*' would need to match -3 characters (impossible).
The fix slices value past parts[0] after the prefix check so that the suffix
search only operates on the remaining substring.
Co-authored-by: JSap0914 <[email protected]>
Co-authored-by: Lukas Taegert-Atkinson <[email protected]>
* Use import attributes for importing JSON * Apply review comments --------- Co-authored-by: Lukas Taegert-Atkinson <[email protected]>
…n filenames (#6408) * fix: insert conflict numbers before first extension in multi-extension filenames makeUnique used node:path extname() which returns only the last extension, so a chunkFileNames pattern like "chunk.d.ts" would produce "chunk.d2.ts" on conflict instead of the correct "chunk2.d.ts". TypeScript declaration chunks generated with rollup-plugin-dts are a common example where this matters because the toolchain requires the full .d.ts extension. Replace the extname() call with a helper that finds the first dot in the filename (skipping a leading dot in hidden-file names) so that all trailing extensions are preserved as a unit when the dedup index is inserted. Fixes #5822 * fix: inline getFirstExtension and fix prettier formatting per review Apply lukastaegert's review comments: inline the getFirstExtension helper directly into makeUnique, working with dotIndex to avoid the Math.max call, and always start the dot search at index 1 instead of conditionally skipping dotfiles. Also fix a prettier line-length violation in the test config. * Ignore dots in directory names and improve coverage --------- Co-authored-by: Lukas Taegert-Atkinson <[email protected]>
* skip side-effect-free external imports when hoisting is off * refactor no-hoist dependency pruning * derive imports from rendered dependencies without mutating this.dependencies The previous approach deleted side-effect-free external dependencies from the long-lived this.dependencies set as a side effect of getRenderedDependencies(). This created a phase-dependent invariant where readers of this.dependencies saw different contents before and after rendering, and could race across concurrent chunk renders (the renderDynamicImport plugin hook reads targetChunk.dependencies during another chunk's renderModules). Instead, leave this.dependencies untouched and derive chunk.imports from getRenderedDependencies().keys(), which already excludes the skipped dependencies. The rendered output and chunk.imports stay in sync with emitted code without mutating the dependency graph. --------- Co-authored-by: Lukas Taegert-Atkinson <[email protected]>
… kind logic to new method (#6416) * refactor(rust/parser_ast/ast_nodes/property): extract write buffer kind logic to new method * refactor(rust/parser_ast/ast_nodes/property): add #[inline] proc macro to write_buffer_kind method --------- Co-authored-by: Lukas Taegert-Atkinson <[email protected]>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
❌ Deploy Preview for relaxed-alfajores-47dd1b failed.
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #213 +/- ##
=======================================
Coverage 98.74% 98.75%
=======================================
Files 274 275 +1
Lines 10787 10822 +35
Branches 2883 2887 +4
=======================================
+ Hits 10652 10687 +35
Misses 92 92
Partials 43 43 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
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
10 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.
此 PR 自动生成。
合并 rollup/rollup 的修改至本仓库（@ 8faa187）
以下文件存在冲突，可能需要重新翻译：
请解决所有冲突后提交到此 pull request，可直接在 Github 修改、切换到 sync-8faa1877 分支进行修改或者使用 Github Desktop 拉取此 pr 修改并提交。
禁止使用 Squash Merge 来合并此 Pull Request！
使用该操作会导致上游仓库的内容丢失，并致使下一次合并产生冲突。