-
Notifications
You must be signed in to change notification settings - Fork 3.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: support react 18 #2515
feat: support react 18 #2515
Conversation
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.
1、不要改 code style
2、不要改版本号
3、peerDependencies 要兼容而不是替换
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.
做最小改动,不要做风格调整。
package.json
Outdated
"repository": "git@github.com:jasonzhuang/dva18.git" | ||
} |
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.
1、去掉 repository
2、结尾得有空行
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.
done
.gitignore
Outdated
@@ -8,3 +8,4 @@ | |||
/packages/**/node_modules | |||
/packages/**/dist | |||
/lerna-debug.log | |||
.history/ |
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.
不要加这个。
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.
我先删掉。但实际上最好有这个说明,不然vscode有些装了这个插件的,会默认生成
packages/dva/src/index.js
Outdated
@@ -20,7 +20,7 @@ const { connectRouter, routerMiddleware } = routerRedux; | |||
const { isFunction } = utils; | |||
const { useHistory, useLocation, useParams, useRouteMatch } = router; | |||
|
|||
export default function(opts = {}) { | |||
export default function (opts = {}) { |
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.
不要做风格调整,应该是你本地的 prettier 和 dva 的 prettier 版本不一致。
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.
done, 暂时把vscode上formatOnSave关闭了解决
.prettierrc
Outdated
@@ -1,5 +1,6 @@ | |||
{ | |||
"printWidth": 100, | |||
"singleQuote": true, | |||
"trailingComma": "all" | |||
"trailingComma": "all", | |||
"semi": true |
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.
不要做调整。
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.
done
Bumped dva@3.0.0-alpha.1 |
升级react版本,在依赖react-dom的地方升级了用法,使用creatRoot来渲染
Close #2514