Skip to content
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

🐛 fix scripts in package.json #1469

Merged
merged 1 commit into from
May 25, 2021
Merged

🐛 fix scripts in package.json #1469

merged 1 commit into from
May 25, 2021

Conversation

gongshun
Copy link
Collaborator

image

@@ -17,7 +17,7 @@
"scripts": {
"examples:install": "npm-run-all --serial build install:*",
"examples:start": "npm-run-all --parallel start:*",
"examples:start-multiple": "run-p start:main-multiple start:react15 start:vue",
"examples:start-multiple": "run-p start-main-multiple start:react15 start:vue",
"install:main": "cd examples/main && yarn",
"start:main": "cd examples/main && yarn start",
"start-main-multiple": "cd examples/main && yarn start:multiple",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

应该把这里改成 start:main-multiple 更好一点,跟其他 script 保持一致

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

如果把 start-main-multiple 改成 start:main-multiple,那么examples:start 命令也会运行 start:main-multiple 的,这是不必要的。最开始写的就是 start:main-multiple,后面被修改了,8c7efcc

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

想起来了~

@@ -17,7 +17,7 @@
"scripts": {
"examples:install": "npm-run-all --serial build install:*",
"examples:start": "npm-run-all --parallel start:*",
"examples:start-multiple": "run-p start:main-multiple start:react15 start:vue",
"examples:start-multiple": "run-p start-main-multiple start:react15 start:vue",
"install:main": "cd examples/main && yarn",
"start:main": "cd examples/main && yarn start",
"start-main-multiple": "cd examples/main && yarn start:multiple",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

想起来了~

@kuitos kuitos merged commit fd289f2 into master May 25, 2021
@kuitos kuitos deleted the gongshun-fix-scripts branch May 25, 2021 09:47
kuitos pushed a commit that referenced this pull request May 29, 2022
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.

None yet

2 participants