-
Notifications
You must be signed in to change notification settings - Fork 2k
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
feat: create-qiankun #2700
feat: create-qiankun #2700
Conversation
🦋 Changeset detectedLatest commit: 1bdeef3 The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
@hakushinring is attempting to deploy a commit to a Personal Account owned by @umijs on Vercel. @umijs first needs to authorize it. |
@kuitos @hakushinring @aladdin-add 各位大佬可以看看第一版功能,是否有东西需要添加,或者修改的 目前完成
待完成
|
template 里好像没看到 react 的主应用? |
package name已经改为 create-qiankun |
@@ -0,0 +1,5 @@ | |||
--- | |||
"@qiankunjs/create-qiankun": patch |
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.
"@qiankunjs/create-qiankun": patch | |
"create-qiankun": patch |
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.
我重新add一下,版本号是不是应该先改成rc版发个测试
那现在如果要发布测试,webpack-plugin react/vue qiankun 这些填啥版本呢。还是等你吧这些全发布后,cli再发布 |
填现在最新的版本就行吧?写 |
最新的lockfile格式是 6.0: https://github.com/pnpm/spec/blob/master/lockfile/6.0.md. 但是这个分支里面是 6.1,不知道什么怎么生成的。🤷 |
发掉了 |
本地 pnpm 版本的问题吧?https://github.com/orgs/pnpm/discussions/6857 |
我本地升级到最新的版本,重新安装也没有变化。可能是之前某个版本的bug,在后来某个patch版本修复掉了,但是之前版本更新过的lockfile被提交了。。 😂 |
可能的解决方案:
|
重新生成一个好了 |
Done |
Co-authored-by: 唯然 <[email protected]>
…/qiankun into feat/create-qiankun
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.
LGTM
是不是可以合了? @hakushinring @qiYuei @aladdin-add |
我觉得可以了,发布个 rc 测试看看 |
qiankun cli 初始化目录