Skip to content

Commit 3e168e4

Browse files
authored
Rework scripts to no longer require the build-tools step, and make eventual direct TS transition easier (#3935)
* move tsconfig.json to dedicated location and switch scripts to build based of rnx-kit's node tsconfig * update tsconfigs to reference new tsconfig.json location * fix some of the base scripts, remove broken debug metro script * rename .ts files in scripts to .js * fix script build errors after full TS validation * use explicit import syntax for local files * move just config to configs folder * convert to module syntax and remove (now unused) just configs * update scripts package to be fully js, use esm, and validate fully * Change files * have scripts use the same command set as all other folders * reorganize scripts and turn on strict linting * update references in babel.config.js files * more script updates * fix lint configuration with new format * fix repo linting with new configs and version, fixing linting errors as well * fix some styling issues from lint --fix and remove unused ignore files * fix jest tests * udpate fixture to remove extra formatting * finish script conversion to modern js with jsdoc syntax * export ObjectBase from framework to help type inferences * exclude lib to not cause false breaks from past builds * remove unnecessary lage config options and leverage the cli for prettier * fix align-deps (which requires .cjs) and remove 'react-dom' preset * remove pipeline step for build tools as well as the repo script * fix check modified task and remove align-deps from depcheck
1 parent 3d36cc9 commit 3e168e4

File tree

365 files changed

+1402
-861
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

365 files changed

+1402
-861
lines changed
Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,6 @@
11
steps:
22
- template: setup-repo.yml
33

4-
- script: |
5-
yarn build-tools
6-
displayName: "yarn build-tools"
7-
84
- script: |
95
yarn build
10-
displayName: "yarn build"
6+
displayName: 'yarn build'

.vscode/launch.json

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -46,21 +46,6 @@
4646
"sourceMaps": true,
4747
"outFiles": []
4848
},
49-
{
50-
"name": "Debug metro bundling task",
51-
"type": "node",
52-
"request": "launch",
53-
"program": "${workspaceRoot}/scripts/debug-metro-pack.js",
54-
"cwd": "${workspaceFolder}/packages/libraries/core",
55-
"args": ["rnTester"],
56-
"runtimeArgs": ["--nolazy"],
57-
"env": {
58-
"NODE_ENV": "development"
59-
},
60-
"console": "integratedTerminal",
61-
"sourceMaps": true,
62-
"outFiles": []
63-
},
6449
{
6550
"name": "Debug Fabric Tester",
6651
"type": "node",

.yarnrc.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,6 @@ nodeLinker: node-modules
77
plugins:
88
- checksum: 672e525b81762c6162366bd3ffec5e86ab8fac2655ef0267047e86a0f32e79a4bde0f170bc30479663f40aa3f006d91f8dc3289f679dd4dc5ae5a5d12ba3ad0b
99
path: .yarn/plugins/@rnx-kit/yarn-plugin-dynamic-extensions.cjs
10-
spec: "https://raw.githubusercontent.com/microsoft/rnx-kit/main/incubator/yarn-plugin-dynamic-extensions/index.js"
10+
spec: 'https://raw.githubusercontent.com/microsoft/rnx-kit/main/incubator/yarn-plugin-dynamic-extensions/index.js'
1111

1212
yarnPath: .yarn/releases/yarn-4.9.2.cjs

apps/E2E/just.config.js

Lines changed: 0 additions & 3 deletions
This file was deleted.

apps/E2E/tsconfig.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"extends": "@fluentui-react-native/scripts/tsconfig.json",
2+
"extends": "@fluentui-react-native/scripts/configs/tsconfig.json",
33
"compilerOptions": {
44
"baseUrl": ".",
55
"importHelpers": true,
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
module.exports = require('@fluentui-react-native/scripts/babel.config');
1+
module.exports = require('@fluentui-react-native/scripts/configs/babel.config');

apps/component-generator/component-templates/ComponentTemplate/tsconfig.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"extends": "@fluentui-react-native/scripts/tsconfig.json",
2+
"extends": "@fluentui-react-native/scripts/configs/tsconfig.json",
33
"compilerOptions": {
44
"outDir": "lib"
55
},

apps/fluent-tester/just.config.js

Lines changed: 0 additions & 3 deletions
This file was deleted.

apps/fluent-tester/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,7 @@
177177
"alignDeps": {
178178
"presets": [
179179
"microsoft/react-native",
180-
"@fluentui-react-native/scripts/align-deps-preset.js"
180+
"@fluentui-react-native/scripts/configs/align-deps-preset.cjs"
181181
],
182182
"requirements": [
183183

apps/fluent-tester/tsconfig.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"extends": "@fluentui-react-native/scripts/tsconfig.json",
2+
"extends": "@fluentui-react-native/scripts/configs/tsconfig.json",
33
"compilerOptions": {
44
"baseUrl": ".",
55
"importHelpers": true,

0 commit comments

Comments
 (0)