-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Upgrade Parcel and other dev dependencies #2701
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
Changes from 27 commits
5a3a830
d5b2b82
edde201
0922260
7c96e3e
8addd50
0da1a3a
74306cb
3a98c05
84c2850
61c4876
090e70b
bc3f9dd
6d7580f
c9669c6
59d226b
325760d
1af2426
f775508
9ed41ba
bbe87d6
fc2b536
7c136b5
d5edb07
1cb77b0
7caa267
a31337f
908b98c
3bcceff
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 |
|---|---|---|
| @@ -1,19 +1,25 @@ | ||
| { | ||
| "extends": "@parcel/config-default", | ||
| "resolvers": ["parcel-resolver-docs", "..."], | ||
| "resolvers": ["@parcel/resolver-glob", "parcel-resolver-docs", "..."], | ||
| "bundler": "@parcel/bundler-experimental", | ||
| "transformers": { | ||
| "apiCheck:*.{js,ts,tsx,json}": ["parcel-transformer-docs"], | ||
| "docs:*.{js,ts,tsx,json}": ["parcel-transformer-docs", "@parcel/transformer-inline"], | ||
| "docs-json:*.{js,ts,tsx,json}": ["parcel-transformer-docs"], | ||
| "*.{md,mdx}": ["parcel-transformer-mdx-docs"], | ||
| "*.svg": ["@parcel/transformer-svg-react"], | ||
| "*.css": ["...", "parcel-transformer-css-env"] | ||
| "*.css": ["...", "parcel-transformer-css-env"], | ||
| "*.{js,mjs,jsm,jsx,es6,cjs,ts,tsx}": [ | ||
|
Member
Author
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 disables Babel from running in Parcel, even though we have a |
||
| "@parcel/transformer-js", | ||
| "@parcel/transformer-react-refresh-wrap" | ||
| ] | ||
| }, | ||
| "namers": ["parcel-namer-docs", "..."], | ||
| "packagers": { | ||
| "*.json": "parcel-packager-docs", | ||
| "*.html": "@parcel/packager-js" | ||
| }, | ||
| "optimizers": { | ||
| "*.js": ["parcel-optimizer-ssg", "..."] | ||
| "*.html": ["parcel-optimizer-ssg", "..."] | ||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,7 +2,7 @@ | |
| "presets": [ | ||
| "@babel/preset-typescript", | ||
| "@babel/preset-react", | ||
| ["@parcel/babel-preset-env", | ||
| ["@babel/preset-env", | ||
| { | ||
| "loose": true | ||
| } | ||
|
|
@@ -12,7 +12,7 @@ | |
| "storybook": { | ||
| "presets": [ | ||
| [ | ||
| "@parcel/babel-preset-env", | ||
| "@babel/preset-env", | ||
| { | ||
| "loose": true, | ||
| "targets": { | ||
|
|
@@ -42,7 +42,7 @@ | |
| }, | ||
| "plugins": [ | ||
| [ | ||
| "@parcel/babel-plugin-transform-runtime", | ||
| "@babel/plugin-transform-runtime", | ||
| { | ||
| "version": "^7.6.2" | ||
| } | ||
|
|
@@ -53,15 +53,6 @@ | |
| "legacy": true | ||
| } | ||
| ], | ||
| [ | ||
| "@babel/plugin-proposal-class-properties", | ||
| { | ||
| "loose": true | ||
| } | ||
| ], | ||
| "@babel/plugin-proposal-export-default-from", | ||
| "@babel/plugin-proposal-export-namespace-from", | ||
| "@babel/plugin-syntax-class-properties", | ||
|
Member
Author
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. These are included in |
||
| "transform-glob-import", | ||
| "babel-plugin-macros" | ||
| ], | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -15,19 +15,19 @@ | |
| ], | ||
| "scripts": { | ||
| "check-types": "tsc", | ||
| "install-17": "yarn add -W react@^17.0.1 react-dom@^17.0.1", | ||
| "install-16": "yarn add -W react@^16.8.0 react-dom@^16.8.0", | ||
| "start": "cross-env NODE_ENV=storybook start-storybook -p 9003 --ci -c '.storybook'", | ||
| "build:storybook": "build-storybook -c .storybook -o dist/$(git rev-parse HEAD)/storybook", | ||
| "build:storybook-17": "build-storybook -c .storybook -o dist/$(git rev-parse HEAD)/storybook-17", | ||
| "build:storybook-16": "build-storybook -c .storybook -o dist/$(git rev-parse HEAD)/storybook-16", | ||
| "start:chromatic": "NODE_ENV=storybook start-storybook -p 9004 --ci -c '.chromatic'", | ||
| "build:chromatic": "build-storybook -c .chromatic -o dist/$(git rev-parse HEAD)/chromatic", | ||
| "start:docs": "PARCEL_WORKER_BACKEND=process DOCS_ENV=dev parcel 'packages/@react-{spectrum,aria,stately}/*/docs/*.mdx' 'packages/dev/docs/pages/**/*.mdx'", | ||
| "build:docs": "PARCEL_WORKER_BACKEND=process DOCS_ENV=staging parcel build 'packages/@react-{spectrum,aria,stately}/*/docs/*.mdx' 'packages/dev/docs/pages/**/*.mdx' --no-scope-hoist", | ||
| "build:docs": "PARCEL_WORKER_BACKEND=process DOCS_ENV=staging parcel build 'packages/@react-{spectrum,aria,stately}/*/docs/*.mdx' 'packages/dev/docs/pages/**/*.mdx'", | ||
| "test": "yarn jest", | ||
| "build": "make build", | ||
| "test:ssr": "yarn jest --config jest.ssr.config.js", | ||
| "ci-test": "yarn jest --maxWorkers=2 && yarn test:ssr --runInBand", | ||
| "ci-test-17": "yarn jest --maxWorkers=2 && yarn test:ssr --runInBand", | ||
| "ci-test-16": "yarn jest --maxWorkers=2 && yarn test:ssr --runInBand", | ||
| "lint": "concurrently \"yarn check-types\" \"eslint packages --ext .js,.ts,.tsx\" \"node scripts/lint-packages.js\"", | ||
| "jest": "node scripts/jest.js", | ||
| "copyrights": "babel-node --presets @babel/env ./scripts/addHeaders.js", | ||
|
|
@@ -56,26 +56,21 @@ | |
| "@babel/cli": "^7.12.10", | ||
| "@babel/core": "^7.12.10", | ||
| "@babel/node": "^7.12.10", | ||
| "@babel/plugin-proposal-class-properties": "^7.12.1", | ||
| "@babel/plugin-proposal-decorators": "^7.12.1", | ||
| "@babel/plugin-proposal-export-default-from": "^7.12.1", | ||
| "@babel/plugin-proposal-export-namespace-from": "^7.12.1", | ||
| "@babel/plugin-proposal-object-rest-spread": "^7.12.1", | ||
| "@babel/plugin-syntax-class-properties": "^7.12.1", | ||
| "@babel/plugin-syntax-decorators": "^7.12.1", | ||
| "@babel/plugin-syntax-object-rest-spread": "^7.8.3", | ||
| "@babel/plugin-transform-async-to-generator": "^7.12.1", | ||
| "@babel/plugin-transform-runtime": "^7.12.10", | ||
| "@babel/preset-env": "^7.12.11", | ||
| "@babel/preset-react": "^7.12.10", | ||
| "@babel/preset-typescript": "^7.12.7", | ||
| "@babel/register": "^7.12.10", | ||
| "@babel/runtime": "^7.12.5", | ||
| "@octokit/rest": "*", | ||
| "@parcel/babel-plugin-transform-runtime": "nightly", | ||
| "@parcel/babel-preset-env": "nightly", | ||
| "@parcel/transformer-inline": "nightly", | ||
| "@parcel/transformer-svg-react": "nightly", | ||
| "@parcel/bundler-experimental": "2.0.0-nightly.2573", | ||
| "@parcel/packager-ts": "2.0.0-nightly.951", | ||
| "@parcel/resolver-glob": "2.0.2-nightly.2573", | ||
| "@parcel/transformer-inline": "2.0.0-nightly.951", | ||
| "@parcel/transformer-svg-react": "2.0.2-nightly.2573", | ||
| "@parcel/transformer-typescript-types": "2.0.0-nightly.951", | ||
| "@spectrum-css/component-builder": "^1.0.0", | ||
| "@spectrum-css/vars": "^2.3.0", | ||
| "@storybook/addon-a11y": "^6.1.10", | ||
|
|
@@ -89,9 +84,9 @@ | |
| "@testing-library/dom": "^8.0.0", | ||
| "@testing-library/jest-dom": "^5.14.1", | ||
| "@testing-library/react": "^12.0.0", | ||
| "@testing-library/react-hooks": "^3.4.1", | ||
| "@testing-library/react-hooks": "^7.0.2", | ||
| "@testing-library/user-event": "^12.1.3", | ||
| "@types/react": "^16.9.23", | ||
| "@types/react": "^17.0.37", | ||
| "@types/storybook__react": "^5.2.1", | ||
| "@typescript-eslint/eslint-plugin": "^4.3.0", | ||
| "@typescript-eslint/parser": "^4.3.0", | ||
|
|
@@ -140,9 +135,10 @@ | |
| "md5": "^2.2.1", | ||
| "nyc": "^10.2.0", | ||
| "p-queue": "^6.2.1", | ||
| "parcel": "nightly", | ||
| "parcel": "2.0.0-nightly.949", | ||
| "patch-package": "^6.2.0", | ||
| "plop": "^2.4.0", | ||
| "postcss": "^7.0.0", | ||
| "postcss-calc": "^6.0.0", | ||
| "postcss-custom-properties": "6.3.1", | ||
| "postcss-dir-pseudo-class": "^5.0.0", | ||
|
|
@@ -157,9 +153,9 @@ | |
| "postcss-svg": "^3.0.0", | ||
| "prop-types": "^15.6.0", | ||
| "raf": "^3.4.0", | ||
| "react": "^16.8.0 || ^17.0.0-rc.1", | ||
| "react": "^17.0.2", | ||
| "react-axe": "^3.0.2", | ||
| "react-dom": "^16.8.0 || ^17.0.0-rc.1", | ||
| "react-dom": "^17.0.2", | ||
| "react-overlays": "0.8.3", | ||
| "react-test-renderer": "^16.9.0", | ||
|
Member
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. should we update this as well?
Member
Author
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. Do we even use it? I don't see any references. And the tests seem to pass? |
||
| "recursive-readdir": "^2.2.2", | ||
|
|
@@ -169,7 +165,7 @@ | |
| "storybook-dark-mode": "^1.0.3", | ||
| "style-loader": "^0.23.1", | ||
| "tempy": "^0.5.0", | ||
| "typescript": "^3.8.3", | ||
| "typescript": "^4.5.2", | ||
| "url-loader": "^1.1.2", | ||
| "walk-object": "^4.0.0", | ||
| "webpack": "^4.44.2", | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,11 +1,11 @@ | ||
| <!-- Copyright 2020 Adobe. All rights reserved. | ||
| {/* Copyright 2020 Adobe. All rights reserved. | ||
|
Member
Author
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. Comment format changed from HTML style to JSX style in MDX v2. |
||
| This file is licensed to you under the Apache License, Version 2.0 (the "License"); | ||
| you may not use this file except in compliance with the License. You may obtain a copy | ||
| of the License at http://www.apache.org/licenses/LICENSE-2.0 | ||
| Unless required by applicable law or agreed to in writing, software distributed under | ||
| the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR REPRESENTATIONS | ||
| OF ANY KIND, either express or implied. See the License for the specific language | ||
| governing permissions and limitations under the License. --> | ||
| governing permissions and limitations under the License. */} | ||
|
|
||
| import {Layout} from '@react-spectrum/docs'; | ||
| export default Layout; | ||
|
|
||
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 fixes the issue with some examples not rendering which we had patched Parcel before, disabling optimization. The new bundler is currently in progress and experimental, but it seems to build our docs correctly.