Fan Zhongxu

Results 12 comments of Fan Zhongxu

I found the cause of the problem. tree-sitter-kotlin regards return/break/continue as expressions instead of statement. ```javascript { jump_expression: $ => choice( prec.left(PREC.RETURN_OR_THROW, seq("throw", $._expression)), prec.left(PREC.RETURN_OR_THROW, seq(choice("return", $._return_at), optional($._expression))), "continue", $._continue_at,...

https://github.com/zxffan/repos/tree/vue2-ssr webpack4+vue2+vuex+vue-router+koa+hot reload

@colinaaa As far as I know, pnpm created multiple hardlinks because it needs to ensure the accuracy of the peer dependencies. > [how-peers-are-resolved](https://pnpm.io/how-peers-are-resolved) In your example, lodash should have no...

@dbartholomae After upgrading to v9 and executing the `rush install/update` command, you will find that the content of the `.rush/temp/shrinkwrap-deps.json` file is incorrect. As you can see, rush is currently...

Calling the fastGlob function will lead to OOM nodejs version: v20.16.0

> @fzxen Do you know what's intention to change this `this.isWorkspaceCompatible` for pnpm9? Right now, when I test it, I found out this logic could not support a empty subspace...

After debugging rush-lib, I found that `pnpmfileSettings.json` will be written twice. First time: https://github.com/microsoft/rushstack/blob/438525caf08bcbdb75d33f6747cc34e050837d46/libraries/rush-lib/src/logic/pnpm/PnpmfileConfiguration.ts#L80-L83 Second time: https://github.com/microsoft/rushstack/blob/438525caf08bcbdb75d33f6747cc34e050837d46/libraries/rush-lib/src/logic/pnpm/SubspacePnpmfileConfiguration.ts#L50-L56 The second write operation will overwrite the first write content. And `pnpmfileSettings.json` is...

Some work is needed to support pnpm v9 in @rushstack/lockfile-explorer. I will take a look this weekend.

> [@fzxen](https://github.com/fzxen) FYI pnpm is now up to version 10. 😄 Pnpmv10 did not upgrade the lockfile version, still using lockfileV9. This may mean it will be easier to upgrade...

When the subspace is not enabled, this issue does not exist. I will raise a PR later to try to fix this issue.