-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
TESTING ONLY - update to React 17 #22326
Changes from all commits
85fac79
1167a03
3fc3f7e
db5a04c
bc31f00
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
{ | ||
"type": "minor", | ||
"comment": "Bump React dependency to 17", | ||
"packageName": "@fluentui/codemods", | ||
"email": "[email protected]", | ||
"dependentChangeType": "patch" | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
{ | ||
"type": "patch", | ||
"comment": "Move react-is to peer dependencies and allow 17", | ||
"packageName": "@fluentui/react-conformance", | ||
"email": "[email protected]", | ||
"dependentChangeType": "patch" | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
{ | ||
"type": "none", | ||
"comment": "Update devDependencies", | ||
"packageName": "@fluentui/react-context-selector", | ||
"email": "[email protected]", | ||
"dependentChangeType": "none" | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
{ | ||
"type": "patch", | ||
"comment": "Use React 17 in export to codepen", | ||
"packageName": "@fluentui/react-docsite-components", | ||
"email": "[email protected]", | ||
"dependentChangeType": "patch" | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
{ | ||
"type": "none", | ||
"comment": "Update devDependencies", | ||
"packageName": "@fluentui/react-icons-mdl2", | ||
"email": "[email protected]", | ||
"dependentChangeType": "none" | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
{ | ||
"type": "none", | ||
"comment": "work around new type issue", | ||
"packageName": "@fluentui/react-utilities", | ||
"email": "[email protected]", | ||
"dependentChangeType": "none" | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -123,7 +123,6 @@ | |
"@types/doctrine": "0.0.5", | ||
"@types/enhanced-resolve": "3.0.7", | ||
"@types/enzyme": "3.10.3", | ||
"@types/enzyme-adapter-react-16": "1.0.3", | ||
"@types/eslint": "7.2.13", | ||
"@types/express": "4.17.2", | ||
"@types/fs-extra": "8.0.1", | ||
|
@@ -136,9 +135,10 @@ | |
"@types/node": "10.17.55", | ||
"@types/node-fetch": "2.5.7", | ||
"@types/prettier": "2.2.3", | ||
"@types/react": "16.14.23", | ||
"@types/react-dom": "16.9.14", | ||
"@types/react-test-renderer": "16.9.5", | ||
"@types/react": "17.0.44", | ||
"@types/react-dom": "17.0.15", | ||
"@types/react-is": "17.0.3", | ||
"@types/react-test-renderer": "17.0.1", | ||
"@types/scheduler": "0.16.2", | ||
"@types/semver": "^5.5.0", | ||
"@types/tmp": "0.2.0", | ||
|
@@ -149,6 +149,7 @@ | |
"@typescript-eslint/eslint-plugin": "4.22.0", | ||
"@typescript-eslint/experimental-utils": "4.22.0", | ||
"@typescript-eslint/parser": "4.22.0", | ||
"@wojtekmaj/enzyme-adapter-react-17": "0.6.7", | ||
"ajv": "8.4.0", | ||
"autoprefixer": "10.2.1", | ||
"babel-jest": "26.6.3", | ||
|
@@ -171,7 +172,6 @@ | |
"enhanced-resolve": "5.8.2", | ||
"enquirer": "2.3.6", | ||
"enzyme": "3.10.0", | ||
"enzyme-adapter-react-16": "1.15.2", | ||
"esbuild-loader": "2.18.0", | ||
"eslint": "7.25.0", | ||
"eslint-config-airbnb": "18.2.1", | ||
|
@@ -217,14 +217,15 @@ | |
"prettier": "2.2.1", | ||
"pretty-bytes": "5.6.0", | ||
"raw-loader": "4.0.2", | ||
"react": "16.14.0", | ||
"react": "17.0.2", | ||
"react-app-polyfill": "2.0.0", | ||
"react-dom": "16.14.0", | ||
"react-test-renderer": "16.14.0", | ||
"react-dom": "17.0.2", | ||
"react-is": "17.0.2", | ||
"react-test-renderer": "17.0.2", | ||
"sass": "1.49.11", | ||
"sass-loader": "12.4.0", | ||
"satisfied": "^1.1.1", | ||
"scheduler": "0.19.1", | ||
"scheduler": "0.20.2", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. scheduler 0.20.2 is used in react-dom 17 |
||
"schema-utils": "3.1.1", | ||
"semver": "^6.2.0", | ||
"storybook-addon-export-to-codesandbox": "0.6.4", | ||
|
@@ -286,8 +287,6 @@ | |
"@typescript-eslint/experimental-utils": "4.22.0", | ||
"@typescript-eslint/parser": "4.22.0", | ||
"@types/jest-axe/axe-core": "4.2.1", | ||
"@types/react": "16.14.23", | ||
"@types/react-dom": "16.9.14", | ||
"eslint": "7.25.0" | ||
}, | ||
"syncpack": { | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,8 +23,8 @@ | |
"module": "dist/es/index.js", | ||
"peerDependencies": { | ||
"@fluentui/react-northstar": "0.54.0", | ||
"react": "^16.8.0", | ||
"react-dom": "^16.8.0" | ||
"react": "^16.8.0 || ^17", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This was already in some of the northstar packages, but I added it to the rest of them that specify react deps as peers |
||
"react-dom": "^16.8.0 || ^17" | ||
}, | ||
"publishConfig": { | ||
"access": "public" | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -16,8 +16,7 @@ | |
"@uifabric/utilities": "^7.32.3", | ||
"classnames": "^2.2.6", | ||
"lodash": "^4.17.15", | ||
"prop-types": "^15.7.2", | ||
"react-is": "^16.6.3" | ||
"prop-types": "^15.7.2" | ||
}, | ||
"devDependencies": { | ||
"@fluentui/eslint-plugin": "*", | ||
|
@@ -36,6 +35,7 @@ | |
"peerDependencies": { | ||
"react": "^16.8.0 || ^17", | ||
"react-dom": "^16.8.0 || ^17", | ||
"react-is": "^16 || ^17", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not 100% sure what the correct/desirable approach is here, but since it appears that |
||
"scheduler": "^0.19.0 || ^0.20.0" | ||
}, | ||
"publishConfig": { | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -16,11 +16,8 @@ | |
"immer": "^9.0.12", | ||
"lodash": "^4.17.15", | ||
"lz-string": "^1.4.4", | ||
"react": "16.14.0", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Moved to peers to match other packages |
||
"react-document-title": "^2.0.3", | ||
"react-dom": "16.14.0", | ||
"react-frame-component": "^4.1.1", | ||
"react-is": "^16.6.3", | ||
"use-immer": "^0.6.0" | ||
}, | ||
"devDependencies": { | ||
|
@@ -30,7 +27,10 @@ | |
"lerna-alias": "^3.0.3-0" | ||
}, | ||
"peerDependencies": { | ||
"@babel/standalone": "^7.8.3" | ||
"@babel/standalone": "^7.8.3", | ||
"react": "^16.8.0 || ^17", | ||
"react-dom": "^16.8.0 || ^17", | ||
"react-is": "^16 || ^17" | ||
}, | ||
"files": [ | ||
"dist" | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,13 +5,11 @@ | |
"author": "Oleksandr Fediashov <[email protected]>", | ||
"bugs": "https://github.com/microsoft/fluentui/issues", | ||
"dependencies": { | ||
"@babel/runtime": "^7.10.4", | ||
"react-is": "^16.6.3" | ||
"@babel/runtime": "^7.10.4" | ||
}, | ||
"devDependencies": { | ||
"@fluentui/eslint-plugin": "*", | ||
"@fluentui/scripts": "^1.0.0", | ||
"@types/react-is": "^16.7.1", | ||
"lerna-alias": "^3.0.3-0" | ||
}, | ||
"files": [ | ||
|
@@ -24,7 +22,8 @@ | |
"module": "dist/es/index.js", | ||
"peerDependencies": { | ||
"react": "^16.8.0 || ^17", | ||
"react-dom": "^16.8.0 || ^17" | ||
"react-dom": "^16.8.0 || ^17", | ||
"react-is": "^16 || ^17" | ||
}, | ||
"publishConfig": { | ||
"access": "public" | ||
|
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.
Updates to this file won't be part of the real change, just making the build pass