Skip to content

Commit 165a466

Browse files
committed
Address feedback from peer review and set Node engine version
1 parent 4561c10 commit 165a466

File tree

14 files changed

+56
-9
lines changed

14 files changed

+56
-9
lines changed

.vscode/extensions.json

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
{
2+
// See https://go.microsoft.com/fwlink/?LinkId=827846
3+
// for the documentation about the extensions.json format
4+
"recommendations": [
5+
"ms-dotnettools.csharp",
6+
"k--kato.docomment"
7+
]
8+
}

docs/BuildFromSource.md

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,12 @@ To get started, you'll need to have a fork of the repo cloned locally. This work
1111
git clone --recursive https://github.com/YOUR_USERNAME/aspnetcore
1212
```
1313

14-
3. If you've already cloned the repo without passing the `--recursive` flag, you can fetch submodule sources at any time using:
15-
16-
```
17-
git submodule update --init --recursive
18-
```
14+
> :bulb: If you've already cloned the repo without passing the `--recursive` flag, you can fetch submodule sources at any time using:
15+
>
16+
> ```
17+
> git submodule update --init --recursive
18+
> ```
19+
>
1920
2021
The experience for building the repo is slightly different based on what environment you are looking to develop in. Select one of the links below to navigate to the instructions for your environment of choice.
2122
@@ -62,8 +63,10 @@ Can't find a solution that has the projects you care about? Feel free to make a
6263

6364
## Visual Studio Code on Windows, Linux, Mac
6465

66+
> :bulb: The instructions below use Visual Studio code as the editor of choice but the same instructions can be used for any other text editor by replacing the `code` command with an invocation to your editor of choice.
67+
6568
1. In order to use Visual Studio Code for development on this repo, you'll need to have [VS Code](https://code.visualstudio.com/) installed and [the `code` command installed](https://code.visualstudio.com/docs/setup/mac#_launching-from-the-command-line) on your machine.
66-
2. The repo constains some JavaScript dependencies, so you will need to install [Node](https://nodejs.org/en/) and [yarn](https://yarnpkg.com/) on your machine.
69+
2. The repo contains some JavaScript dependencies, so you will need to install [Node](https://nodejs.org/en/) and [yarn](https://yarnpkg.com/) on your machine.
6770
3. Prior to opening the code in Visual Studio code, you'll need to run the `restore` script locally to install the required dotnet dependencies and setup the repo. The `restore` script is located in the root of the repo.
6871

6972
```bash

src/Components/Web.JS/package.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,5 +43,8 @@
4343
"resolutions": {
4444
"**/set-value": "^3.0.2",
4545
"url-parse": ">=1.5.0"
46+
},
47+
"engines": {
48+
"node": "16.x"
4649
}
4750
}

src/Components/WebAssembly/Authentication.Msal/src/Interop/package.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,5 +27,8 @@
2727
},
2828
"dependencies": {
2929
"@azure/msal-browser": "^2.16.1"
30+
},
31+
"engines": {
32+
"node": "16.x"
3033
}
3134
}

src/Components/WebAssembly/WebAssembly.Authentication/src/Interop/package.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,5 +27,8 @@
2727
},
2828
"dependencies": {
2929
"oidc-client": "^1.11.5"
30+
},
31+
"engines": {
32+
"node": "16.x"
3033
}
3134
}

src/Components/test/E2ETest/package.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,5 +19,8 @@
1919
"@types/node": "^13.1.7",
2020
"ts-node": "^8.6.2",
2121
"typescript": "^3.7.5"
22+
},
23+
"engines": {
24+
"node": "16.x"
2225
}
2326
}

src/JSInterop/Microsoft.JSInterop.JS/src/package.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,5 +29,8 @@
2929
"rimraf": "^3.0.2",
3030
"tslint": "^6.1.3",
3131
"typescript": "^4.2.2"
32+
},
33+
"engines": {
34+
"node": "16.x"
3235
}
3336
}

src/ProjectTemplates/BlazorTemplates.Tests/package.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,5 +12,8 @@
1212
"license": "MIT",
1313
"dependencies": {
1414
"selenium-standalone": "^7.1.0"
15+
},
16+
"engines": {
17+
"node": "16.x"
1518
}
1619
}

src/ProjectTemplates/test/package.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,5 +12,8 @@
1212
"license": "MIT",
1313
"dependencies": {
1414
"selenium-standalone": "^7.1.0"
15+
},
16+
"engines": {
17+
"node": "16.x"
1518
}
1619
}

src/SignalR/clients/ts/FunctionalTests/package.json

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,5 +57,8 @@
5757
"url-parse": ">=1.5.0"
5858
},
5959
"author": "",
60-
"license": "MIT"
60+
"license": "MIT",
61+
"engines": {
62+
"node": "16.x"
63+
}
6164
}

src/SignalR/clients/ts/common/package.json

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,5 +30,8 @@
3030
"webpack": "^5.24.1",
3131
"webpack-cli": "^4.5.0"
3232
},
33-
"dependencies": {}
33+
"dependencies": {},
34+
"engines": {
35+
"node": "16.x"
36+
}
3437
}

src/SignalR/clients/ts/package.json

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,5 +12,8 @@
1212
"coverage": "node ./common/node_modules/jest/bin/jest.js --config ./jest.config.js --coverage"
1313
},
1414
"author": "Microsoft",
15-
"license": "MIT"
15+
"license": "MIT",
16+
"engines": {
17+
"node": "16.x"
18+
}
1619
}

src/SignalR/clients/ts/signalr-protocol-msgpack/package.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,5 +50,8 @@
5050
"devDependencies": {},
5151
"resolutions": {
5252
"url-parse": ">=1.5.0"
53+
},
54+
"engines": {
55+
"node": "16.x"
5356
}
5457
}

src/SignalR/clients/ts/signalr/package.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,5 +59,8 @@
5959
"fetch-cookie": "^0.11.0",
6060
"node-fetch": "^2.6.1",
6161
"ws": "^7.4.5"
62+
},
63+
"engines": {
64+
"node": "16.x"
6265
}
6366
}

0 commit comments

Comments
 (0)