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

[heft-web-rig] Add an "app" profile, upgrade to Webpack 5, implement new CSS rules #3204

Merged
merged 43 commits into from
Feb 11, 2022
Merged
Show file tree
Hide file tree
Changes from 23 commits
Commits
Show all changes
43 commits
Select commit Hold shift + click to select a range
ba04506
Bring over prototype of updated heft-web-rig
octogonz Feb 4, 2022
05a01b4
Sync some changes to heft-node-rig
octogonz Feb 4, 2022
2e691b6
Relink workspaces dependencies
octogonz Feb 4, 2022
44827a2
Sync dependencies
octogonz Feb 5, 2022
bd7212a
rush update --full
octogonz Feb 5, 2022
fd483cc
Fix a Jest test that failed on Windows OS
octogonz Feb 5, 2022
74ec89a
Reenable strictPeerDependencies=true, upgrade PNPM, and solve all the…
octogonz Feb 5, 2022
f0a227e
rush update --full
octogonz Feb 5, 2022
99777c9
Fix up projects so everything builds successfully
octogonz Feb 5, 2022
9b98c7e
Merge remote-tracking branch 'remotes/origin/master' into octogonz/he…
octogonz Feb 5, 2022
4b3547e
rush change
octogonz Feb 5, 2022
c458420
rush change
octogonz Feb 5, 2022
4728061
Merge remote-tracking branch 'remotes/origin/master' into octogonz/he…
octogonz Feb 7, 2022
b00dbe3
PR feedback -- we're not ready to release 1.0.0 yet
octogonz Feb 7, 2022
2824e3c
PR feedback rush-project.json "outputFolderNames" should match heft.j…
octogonz Feb 7, 2022
02e3346
Update rigs/heft-web-rig/profiles/library/config/typescript.json
octogonz Feb 7, 2022
28b938b
Update rigs/heft-web-rig/profiles/app/config/typescript.json
octogonz Feb 7, 2022
6295160
Update rigs/heft-web-rig/shared/webpack-base.config.js
octogonz Feb 7, 2022
7baea3b
Merge branch 'octogonz/heft-web-rig-revamp' of https://github.com/mic…
octogonz Feb 7, 2022
9fa72da
Convert "webpack" to be a peer dependency of "heft-webpack?-plugin". …
octogonz Feb 7, 2022
d6eca76
rush update --full
octogonz Feb 7, 2022
2f44260
rush change
octogonz Feb 7, 2022
7c0aa06
rush rebuild
octogonz Feb 7, 2022
deee2f2
Enable (optional) bundling of library projects
octogonz Feb 7, 2022
57a632d
Add test projects
octogonz Feb 9, 2022
b70867b
rush update
octogonz Feb 9, 2022
e8f1ae8
Fix some issues with the Webpack config
octogonz Feb 9, 2022
a1bab5f
Example app now builds an runs
octogonz Feb 9, 2022
198b595
PR feedback - cosmetic issues
octogonz Feb 9, 2022
5436f15
Regenerate README.md
octogonz Feb 9, 2022
770ff15
PR feedback -- remove @types/webpack-dev-server
octogonz Feb 9, 2022
4d05712
Show how to configure heft-web-rig-app-tutorial to consume the librar…
octogonz Feb 9, 2022
4fed4d3
PR feedback: Replace "file-loader" with asset modules, and update the…
octogonz Feb 11, 2022
9aeadf2
rush update
octogonz Feb 11, 2022
1c3388d
Move src/templates -> src/schemas/templates to make this folder more …
octogonz Feb 11, 2022
f04d5f7
Bring back the template for sass.json, which got inadvertently delete…
octogonz Feb 11, 2022
72ad161
Replace sass-loader with heft-sass-plugin (work in progress, NOT WORK…
octogonz Feb 11, 2022
bccec15
Temporarily revert the sass.json features from 9aeadf28d49c6a9dc19811…
octogonz Feb 11, 2022
5ccbe25
Fix an issue where source maps weren't loaded for library projects
octogonz Feb 11, 2022
da1d200
rush change
octogonz Feb 11, 2022
bb0edf2
Merge remote-tracking branch 'remotes/origin/master' into octogonz/he…
octogonz Feb 11, 2022
06f121a
rush update --full
octogonz Feb 11, 2022
55ef407
Update README.md
octogonz Feb 11, 2022
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -11,26 +11,27 @@
"_phase:build": ""
},
"devDependencies": {
"@babel/core": "^7.14.8",
"@storybook/addon-actions": "~6.3.6",
"@storybook/addon-essentials": "~6.3.6",
"@storybook/addon-links": "~6.3.6",
"@storybook/cli": "~6.3.6",
"@storybook/components": "~6.3.0",
"@storybook/core-events": "~6.3.0",
"@storybook/react": "~6.3.6",
"@storybook/theming": "~6.3.0",
"@babel/core": "~7.17.0",
"@storybook/addon-actions": "~6.4.18",
"@storybook/addon-essentials": "~6.4.18",
"@storybook/addon-links": "~6.4.18",
"@storybook/cli": "~6.4.18",
"@storybook/components": "~6.4.18",
"@storybook/core-events": "~6.4.18",
"@storybook/react": "~6.4.18",
"@storybook/theming": "~6.4.18",
"@types/heft-jest": "1.0.1",
"@types/react-dom": "16.9.8",
"@types/react": "16.9.45",
"@types/react-dom": "16.9.14",
"@types/react": "16.14.23",
"@types/webpack-env": "1.13.0",
"babel-loader": "^8.2.2",
"css-loader": "~4.2.1",
"babel-loader": "~8.2.3",
"css-loader": "~5.2.7",
"jest": "~27.4.2",
"react-dom": "~16.13.1",
"react": "~16.13.1",
"style-loader": "~1.2.1",
"style-loader": "~2.0.0",
"typescript": "~4.5.2",
"webpack": "~4.44.2"
"webpack": "~4.44.2",
"terser-webpack-plugin": "~3.0.8"
octogonz marked this conversation as resolved.
Show resolved Hide resolved
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
/**
* The path to the plugin package.
*/
"plugin": "@rushstack/heft-webpack5-plugin"
"plugin": "@rushstack/heft-webpack4-plugin"

/**
* An optional object that provides additional settings that may be defined by the plugin.
Expand Down
15 changes: 8 additions & 7 deletions build-tests-samples/heft-storybook-react-tutorial/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,27 +6,28 @@
"scripts": {
"build": "heft build --clean",
"start": "heft start",
"storybook": "heft start --storybook",
"_phase:build": "heft build --clean",
"_phase:test": "heft test --no-build"
},
"devDependencies": {
"@rushstack/eslint-config": "workspace:*",
"@rushstack/heft-jest-plugin": "workspace:*",
"@rushstack/heft-storybook-plugin": "workspace:*",
"@rushstack/heft-webpack5-plugin": "workspace:*",
"@rushstack/heft-webpack4-plugin": "workspace:*",
iclanton marked this conversation as resolved.
Show resolved Hide resolved
"@rushstack/heft": "workspace:*",
"@types/heft-jest": "1.0.1",
"@types/react-dom": "16.9.8",
"@types/react": "16.9.45",
"@types/react-dom": "16.9.14",
"@types/react": "16.14.23",
"@types/webpack-env": "1.13.0",
"css-loader": "~4.2.1",
"css-loader": "~5.2.7",
"eslint": "~8.7.0",
"heft-storybook-react-tutorial-storykit": "workspace:*",
"html-webpack-plugin": "~5.5.0",
"html-webpack-plugin": "~4.5.2",
"react-dom": "~16.13.1",
"react": "~16.13.1",
"source-map-loader": "~1.1.2",
"style-loader": "~1.2.1",
"source-map-loader": "~1.1.3",
"style-loader": "~2.0.0",
"typescript": "~4.5.2",
"webpack": "~4.44.2"
}
Expand Down
12 changes: 6 additions & 6 deletions build-tests-samples/heft-webpack-basic-tutorial/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,17 +15,17 @@
"@rushstack/heft-jest-plugin": "workspace:*",
"@rushstack/heft-webpack5-plugin": "workspace:*",
"@types/heft-jest": "1.0.1",
"@types/react": "16.9.45",
"@types/react-dom": "16.9.8",
"@types/react": "16.14.23",
"@types/react-dom": "16.9.14",
octogonz marked this conversation as resolved.
Show resolved Hide resolved
"@types/webpack-env": "1.13.0",
"css-loader": "~4.2.1",
"css-loader": "~6.6.0",
"eslint": "~8.7.0",
"html-webpack-plugin": "~5.5.0",
"react": "~16.13.1",
"react-dom": "~16.13.1",

Choose a reason for hiding this comment

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

Suggested change
"react": "~16.13.1",
"react-dom": "~16.13.1",
"react-dom": "~16.13.1",
"react": "~16.13.1",

"style-loader": "~1.2.1",
"style-loader": "~3.3.1",
"typescript": "~4.5.2",
"webpack": "~4.44.2",
"source-map-loader": "~1.1.2"
"webpack": "~5.68.0",
"source-map-loader": "~3.0.1"
octogonz marked this conversation as resolved.
Show resolved Hide resolved
}
}
2 changes: 1 addition & 1 deletion build-tests/heft-sass-test/config/heft.json
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
/**
* The path to the plugin package.
*/
"plugin": "@rushstack/heft-webpack5-plugin"
"plugin": "@rushstack/heft-webpack4-plugin"

/**
* An optional object that provides additional settings that may be defined by the plugin.
Expand Down
22 changes: 11 additions & 11 deletions build-tests/heft-sass-test/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,22 +14,22 @@
"@rushstack/heft": "workspace:*",
"@rushstack/heft-jest-plugin": "workspace:*",
"@rushstack/heft-sass-plugin": "workspace:*",
"@rushstack/heft-webpack5-plugin": "workspace:*",
"@rushstack/heft-webpack4-plugin": "workspace:*",
iclanton marked this conversation as resolved.
Show resolved Hide resolved
"@types/heft-jest": "1.0.1",
"@types/react-dom": "16.9.8",
"@types/react": "16.9.45",
"@types/react-dom": "16.9.14",
"@types/react": "16.14.23",
"@types/webpack-env": "1.13.0",
"autoprefixer": "~9.8.0",
"css-loader": "~4.2.1",
"autoprefixer": "~10.4.2",
"css-loader": "~5.2.7",
"eslint": "~8.7.0",
"html-webpack-plugin": "~5.5.0",
"node-sass": "6.0.1",
"postcss": "7.0.32",
"postcss-loader": "~4.0.1",
"html-webpack-plugin": "~4.5.2",
"postcss": "~8.4.6",
"postcss-loader": "~4.1.0",
octogonz marked this conversation as resolved.
Show resolved Hide resolved
"react-dom": "~16.13.1",
"react": "~16.13.1",
"sass-loader": "~12.3.0",
"style-loader": "~1.2.1",
"sass": "~1.3.0",
"sass-loader": "~10.0.0",
octogonz marked this conversation as resolved.
Show resolved Hide resolved
"style-loader": "~2.0.0",
"typescript": "~4.5.2",
"webpack": "~4.44.2"
},
Expand Down
1 change: 0 additions & 1 deletion build-tests/heft-sass-test/webpack.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
const path = require('path');
const Autoprefixer = require('autoprefixer');
const HtmlWebpackPlugin = require('html-webpack-plugin');
const sass = require('node-sass');

/**
* If the "--production" command-line parameter is specified when invoking Heft, then the
Expand Down
25 changes: 13 additions & 12 deletions build-tests/install-test-workspace/workspace/common/pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions build-tests/localization-plugin-test-01/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,11 @@
"@rushstack/node-core-library": "workspace:*",
"@rushstack/set-webpack-public-path-plugin": "workspace:*",
"@types/webpack-env": "1.13.0",
"html-webpack-plugin": "~4.5.0",
"html-webpack-plugin": "~4.5.2",
"ts-loader": "6.0.0",
"typescript": "~4.5.2",
"webpack": "~4.44.2",
"webpack-bundle-analyzer": "~3.6.0",
"webpack-bundle-analyzer": "~4.5.0",
"webpack-cli": "~3.3.2",
"webpack-dev-server": "~3.11.0"
}
Expand Down
4 changes: 2 additions & 2 deletions build-tests/localization-plugin-test-02/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,12 @@
"@rushstack/set-webpack-public-path-plugin": "workspace:*",
"@types/lodash": "4.14.116",
"@types/webpack-env": "1.13.0",
"html-webpack-plugin": "~4.5.0",
"html-webpack-plugin": "~4.5.2",
"lodash": "~4.17.15",
"ts-loader": "6.0.0",
"typescript": "~4.5.2",
"webpack": "~4.44.2",
"webpack-bundle-analyzer": "~3.6.0",
"webpack-bundle-analyzer": "~4.5.0",
"webpack-cli": "~3.3.2",
"webpack-dev-server": "~3.11.0"
octogonz marked this conversation as resolved.
Show resolved Hide resolved
}
Expand Down
4 changes: 2 additions & 2 deletions build-tests/localization-plugin-test-03/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,11 @@
"@rushstack/node-core-library": "workspace:*",
"@rushstack/set-webpack-public-path-plugin": "workspace:*",
"@types/webpack-env": "1.13.0",
"html-webpack-plugin": "~4.5.0",
"html-webpack-plugin": "~4.5.2",
"ts-loader": "6.0.0",
"typescript": "~4.5.2",
"webpack": "~4.44.2",
"webpack-bundle-analyzer": "~3.6.0",
"webpack-bundle-analyzer": "~4.5.0",
"webpack-cli": "~3.3.2",
"webpack-dev-server": "~3.11.0"
octogonz marked this conversation as resolved.
Show resolved Hide resolved
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,10 @@
"@rushstack/heft": "workspace:*",
"@rushstack/heft-webpack4-plugin": "workspace:*",
"@rushstack/set-webpack-public-path-plugin": "workspace:*",
"html-webpack-plugin": "~4.5.0",
"html-webpack-plugin": "~4.5.2",
"@types/webpack-env": "1.13.0",
"eslint": "~8.7.0",
octogonz marked this conversation as resolved.
Show resolved Hide resolved
"typescript": "~4.5.2"
"typescript": "~4.5.2",
"webpack": "~4.44.2"
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
{
"changes": [
{
"packageName": "@microsoft/load-themed-styles",
"comment": "Enable importHelpers=true",
"type": "patch"
}
],
"packageName": "@microsoft/load-themed-styles"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
{
"changes": [
{
"packageName": "@microsoft/rush",
"comment": "",
"type": "none"
}
],
"packageName": "@microsoft/rush"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
{
"changes": [
{
"packageName": "@rushstack/heft-dev-cert-plugin",
"comment": "",
"type": "none"
}
],
"packageName": "@rushstack/heft-dev-cert-plugin"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
{
"changes": [
{
"packageName": "@rushstack/heft-node-rig",
"comment": "Set useUnknownInCatchVariables=false in tsconfig.json, to work around https://github.com/microsoft/TypeScript/issues/42596",
"type": "minor"
}
],
"packageName": "@rushstack/heft-node-rig"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
{
"changes": [
{
"packageName": "@rushstack/heft-sass-plugin",
"comment": "Upgrade postcss",
"type": "patch"
}
],
"packageName": "@rushstack/heft-sass-plugin"
}
Loading