Skip to content

Commit

Permalink
build: Upgrade pipelines to Node18 + upgrade nvm recommendation (#17389)
Browse files Browse the repository at this point in the history
(Cherry pick of a0368d6)

Update the recommended versions of Node.js for developers to use, as
well as the version used by CI, from version 14 to 18.

This change made some scenarios in some e2e tests hang, so a fix for
skipping them had to be included.
Details:
0.58.x doesn't have the fix to routerlicious-driver in this PR:
#8913
(also see related issue for more context on the problem:
#9163)
This causes this test to hang while loading container2, as the snapshot
is over 16KB.
  • Loading branch information
robertobe-ms authored and alexvy86 committed Feb 23, 2024
1 parent 95aa664 commit 232921a
Show file tree
Hide file tree
Showing 58 changed files with 17,352 additions and 17,594 deletions.
2 changes: 1 addition & 1 deletion .nvmrc
Original file line number Diff line number Diff line change
@@ -1 +1 @@
14
18
2 changes: 1 addition & 1 deletion PACKAGES.md
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ The dependencies between layers are enforced by the layer-check command._

| Packages | Layer Dependencies |
| --- | --- |
| - [@fluidframework/azure-local-service](/server/azure-local-service)</br>- [tinylicious](/server/tinylicious)</br>&nbsp;</br>&nbsp;</br>&nbsp;</br>&nbsp; | - [Base-Definitions](#Base-Definitions)</br>- [Protocol-Definitions](#Protocol-Definitions)</br>- [Base-Utils](#Base-Utils)</br>- [Protocol-Utils](#Protocol-Utils)</br>- [Server-Shared-Utils](#Server-Shared-Utils)</br>- [Server-Libs](#Server-Libs) |
| - [@fluidframework/azure-local-service](/azure/packages/azure-local-service)</br>- [tinylicious](/server/tinylicious)</br>&nbsp;</br>&nbsp;</br>&nbsp;</br>&nbsp; | - [Base-Definitions](#Base-Definitions)</br>- [Protocol-Definitions](#Protocol-Definitions)</br>- [Base-Utils](#Base-Utils)</br>- [Protocol-Utils](#Protocol-Utils)</br>- [Server-Shared-Utils](#Server-Shared-Utils)</br>- [Server-Libs](#Server-Libs) |

### Routerlicious-Driver

Expand Down
11 changes: 4 additions & 7 deletions examples/apps/spaces/tsconfig.json
Original file line number Diff line number Diff line change
@@ -1,14 +1,13 @@
{
"extends": "@fluidframework/build-common/ts-common-config.json",
"exclude": [
"dist",
"node_modules"
],
"exclude": ["dist", "node_modules"],
"compilerOptions": {
"target": "es6",
"declarationDir": "./dist",
"outDir": "./dist",
"jsx": "react",
// Added because upgrading to node 18 caused a bunch of typecheck errors
"skipLibCheck": true,
"types": [
"jest",
"puppeteer",
Expand All @@ -18,7 +17,5 @@
"react-dom"
]
},
"include": [
"src/**/*"
]
"include": ["src/**/*"]
}
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ interface IDiceRollerViewProps {
model: IDiceRoller;
}

export const DiceRollerView: React.FC<IDiceRollerViewProps> = (props: IDiceRollerViewProps) => {
export const DiceRollerView = (props: IDiceRollerViewProps) => {
const [diceValue, setDiceValue] = React.useState(props.model.value);

React.useEffect(() => {
Expand Down
2 changes: 2 additions & 0 deletions examples/apps/view-framework-sampler/tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
"module": "esnext",
"outDir": "dist",
"jsx": "react",
// Added because upgrading to node 18 caused a bunch of typecheck errors
"skipLibCheck": true,
"types": [
"jest",
"puppeteer",
Expand Down
26 changes: 13 additions & 13 deletions examples/data-objects/prosemirror/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -56,20 +56,20 @@
"@fluidframework/sequence": "^1.4.0",
"@fluidframework/view-adapters": "^1.4.0",
"@fluidframework/view-interfaces": "^1.4.0",
"@types/prosemirror-model": "^1.7.2",
"@types/prosemirror-schema-list": "^1.0.1",
"@types/prosemirror-state": "^1.2.3",
"@types/prosemirror-view": "^1.9.0",
"@types/prosemirror-model": "1.16.2",
"@types/prosemirror-schema-list": "1.0.3",
"@types/prosemirror-state": "1.3.0",
"@types/prosemirror-view": "1.23.3",
"orderedmap": "^1.1.1",
"prosemirror-example-setup": "^1.0.1",
"prosemirror-history": "^1.1.3",
"prosemirror-keymap": "^1.1.3",
"prosemirror-menu": "^1.2.1",
"prosemirror-model": "^1.7.2",
"prosemirror-schema-list": "^1.0.3",
"prosemirror-state": "^1.2.4",
"prosemirror-transform": "^1.2.3",
"prosemirror-view": "^1.10.3"
"prosemirror-example-setup": "1.1.2",
"prosemirror-history": "1.3.0",
"prosemirror-keymap": "1.2.0",
"prosemirror-menu": "1.2.1",
"prosemirror-model": "1.17.0",
"prosemirror-schema-list": "1.2.0",
"prosemirror-state": "1.4.0",
"prosemirror-transform": "1.6.0",
"prosemirror-view": "1.25.0"
},
"devDependencies": {
"@fluid-tools/webpack-fluid-loader": "^1.4.0",
Expand Down
2 changes: 2 additions & 0 deletions examples/data-objects/vltava/tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@
"module": "esnext",
"outDir": "dist",
"jsx": "react",
// Added because upgrading to node 18 caused a bunch of typechecks errors
"skipLibCheck": true,
"types": [
"jest",
"puppeteer",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@
"cross-env": "^7.0.2",
"eslint": "~8.6.0",
"mocha": "^10.0.0",
"nock": "^10.0.1",
"nock": "^13.3.3",
"nyc": "^15.0.0",
"rimraf": "^2.6.2",
"sinon": "^7.4.2",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@
"cross-env": "^7.0.2",
"eslint": "~8.6.0",
"mocha": "^10.0.0",
"nock": "^10.0.1",
"nock": "^13.3.3",
"nyc": "^15.0.0",
"rimraf": "^2.6.2",
"sinon": "^7.4.2",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@
"cross-env": "^7.0.2",
"eslint": "~8.6.0",
"mocha": "^10.0.0",
"nock": "^10.0.1",
"nock": "^13.3.3",
"nyc": "^15.0.0",
"rimraf": "^2.6.2",
"sinon": "^7.4.2",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
"chai": "^4.2.0",
"cross-env": "^7.0.2",
"mocha": "^10.0.0",
"nock": "^10.0.1",
"nock": "^13.3.3",
"nyc": "^15.0.0",
"sinon": "^7.4.2"
}
Expand Down
11 changes: 0 additions & 11 deletions experimental/dds/tree-graphql/.env-cmdrc.json

This file was deleted.

83 changes: 0 additions & 83 deletions experimental/dds/tree-graphql/.eslintrc.js

This file was deleted.

18 changes: 0 additions & 18 deletions experimental/dds/tree-graphql/.gitignore

This file was deleted.

6 changes: 0 additions & 6 deletions experimental/dds/tree-graphql/.npmignore

This file was deleted.

8 changes: 0 additions & 8 deletions experimental/dds/tree-graphql/.nycrc.json

This file was deleted.

18 changes: 0 additions & 18 deletions experimental/dds/tree-graphql/.prettierignore

This file was deleted.

8 changes: 0 additions & 8 deletions experimental/dds/tree-graphql/.prettierrc.json

This file was deleted.

21 changes: 0 additions & 21 deletions experimental/dds/tree-graphql/LICENSE

This file was deleted.

11 changes: 0 additions & 11 deletions experimental/dds/tree-graphql/README.md

This file was deleted.

4 changes: 0 additions & 4 deletions experimental/dds/tree-graphql/api-extractor.json

This file was deleted.

6 changes: 0 additions & 6 deletions experimental/dds/tree-graphql/codegen.yml

This file was deleted.

Loading

0 comments on commit 232921a

Please sign in to comment.