Skip to content

chore: fix types #19772

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

Merged
merged 3 commits into from
Aug 6, 2025
Merged

chore: fix types #19772

merged 3 commits into from
Aug 6, 2025

Conversation

alexander-akait
Copy link
Member

What kind of change does this PR introduce?

bugfix types

Did you add tests for your changes?

Existing

Does this PR introduce a breaking change?

No

What needs to be documented once your changes are merged?

Nothing

Copy link

codspeed-hq bot commented Aug 6, 2025

CodSpeed Performance Report

Merging #19772 will degrade performances by 49.66%

Comparing chore-fix-types (ba0f5dd) with main (af76497)

Summary

⚡ 1 improvements
❌ 1 regressions
✅ 31 untouched benchmarks
⁉️ 9 dropped benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark BASE HEAD Change
benchmark "cache-filesystem", scenario '{"name":"mode-development-rebuild","mode":"development","watch":true}' 5,564.7 ms 348.8 ms ×16
benchmark "devtool-eval", scenario '{"name":"mode-development-rebuild","mode":"development","watch":true}' 44.7 ms 88.7 ms -49.66%
⁉️ benchmark "lodash", scenario '{"name":"mode-development","mode":"development"}' 774.4 ms N/A N/A
⁉️ benchmark "lodash", scenario '{"name":"mode-development-rebuild","mode":"development","watch":true}' 74.2 ms N/A N/A
⁉️ benchmark "lodash", scenario '{"name":"mode-production","mode":"production"}' 9.5 s N/A N/A
⁉️ benchmark "many-chunks-commonjs", scenario '{"name":"mode-development","mode":"development"}' 267.9 ms N/A N/A
⁉️ benchmark "many-chunks-commonjs", scenario '{"name":"mode-development-rebuild","mode":"development","watch":true}' 94.1 ms N/A N/A
⁉️ benchmark "many-chunks-commonjs", scenario '{"name":"mode-production","mode":"production"}' 2 s N/A N/A
⁉️ benchmark "many-chunks-esm", scenario '{"name":"mode-development","mode":"development"}' 256.5 ms N/A N/A
⁉️ benchmark "many-chunks-esm", scenario '{"name":"mode-development-rebuild","mode":"development","watch":true}' 98.2 ms N/A N/A
⁉️ benchmark "many-chunks-esm", scenario '{"name":"mode-production","mode":"production"}' 2.3 s N/A N/A

@alexander-akait alexander-akait merged commit 67379fe into main Aug 6, 2025
42 of 43 checks passed
@alexander-akait alexander-akait deleted the chore-fix-types branch August 6, 2025 13:03
colinaaa added a commit to lynx-family/lynx-stack that referenced this pull request Aug 7, 2025
Some pnpm-related tweaks:

1. Upgrade to `pnpm@10.14.0`.
2. Fix `webpack@5.99.9` patch failed since we have `ignorePatchFailures:
false` (the patch has been merged in
webpack/webpack#19772, so we can remove it from
next webpack release).
3. ~~Add `typescript` to the `peerDependencies` of `ts-patch` for
enableGlobalVirtualStore. This would solve the "Error: Cannot find
module 'typescript'" error during local `pnpm install`.~~ We disabled
`enableGlobalVirtualStore` for now. ReactLynx needs `preact` and
`@lynx-js/react` to be hoisted.
4. Remove useless overrides to `path-serializer`.
5. Remove `@pnpm/plugin-better-defaults` due to
pnpm/plugin-better-defaults#2

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Chores**
  * Updated the package manager version to pnpm 10.14.0.
* Improved workspace configuration for dependency management, including
more precise patching and enhanced compatibility for certain packages.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

## Checklist

<!--- Check and mark with an "x" -->

- [ ] Tests updated (or not required).
- [ ] Documentation updated (or not required).
- [ ] Changeset added, and when a BREAKING CHANGE occurs, it needs to be
clearly marked (or not required).
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.

1 participant