-
Notifications
You must be signed in to change notification settings - Fork 299
docs: added configuration examples to improve all TinyVue related dependencies to solve the problem that pnpm projects need to install multiple sub-packages #3377
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
base: dev
Are you sure you want to change the base?
Conversation
WalkthroughA new configuration recommendation was added for pnpm-based projects in the documentation. The update suggests including a Changes
Poem
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Walkthrough在 Changes
|
@@ -73,6 +73,13 @@ module.exports = { | |||
} | |||
``` | |||
|
|||
或者,你也可以在项目根目录的 `.npmrc` 文件中添加以下配置来提升所有 TinyVue 相关的依赖: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This addition provides a configuration example to enhance all TinyVue-related dependencies by modifying the .npmrc
file. It addresses the issue where pnpm projects require multiple sub-packages to be installed. Ensure that this configuration is compatible with existing project setups.
WalkthroughAdded a configuration example in Changes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (3)
examples/sites/demos/pc/webdoc/import-components.md (3)
76-82
: Consider linking to pnpm's official documentation
To help users understand and experiment withpublic-hoist-pattern
, add a link to the pnpm docs (e.g., https://pnpm.io/npmrc#public-hoist-pattern). This gives more context on how this setting works and what other options are available.
76-82
: Clarify supported pnpm versions
It may be helpful to mention that thepublic-hoist-pattern
setting requires pnpm v7+ (or whichever minimum version your project supports) to avoid confusion for users on older pnpm releases.
78-81
: Usenpmrc
code fence for better syntax highlighting
Switch the snippet’s fence fromini to
npmrc so editors and docs generators apply proper syntax highlighting for a.npmrc
file.
在 import-components.md 中添加提升所有 TinyVue 相关依赖的配置示例,解决pnpm工程需要安装多个子包的问题
PR
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
Other information
Summary by CodeRabbit
.npmrc
file.