Skip to content

Commit cee0470

Browse files
madskristiansenmkArtakMSFT
authored andcommitted
Fixed a few typos in BuildFromSource documentation (#14624)
1 parent 97db493 commit cee0470

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

docs/BuildFromSource.md

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
Build ASP.NET Core from Source
22
==============================
33

4-
Building ASP.NET Core from source allows you tweak and customize ASP.NET Core, and to contribute your improvements back to the project.
4+
Building ASP.NET Core from source allows you to tweak and customize ASP.NET Core, and to contribute your improvements back to the project.
55

66
See https://github.com/aspnet/AspNetCore/labels/area-infrastructure for known issues and to track ongoing work.
77

8-
## Install pre-requistes
8+
## Install pre-requisites
99

1010
### Windows
1111

@@ -44,7 +44,7 @@ Building ASP.NET Core on macOS or Linux requires:
4444
4545
## Clone the source code
4646
47-
ASP.NET Core uses git submodules to include source from a few other projects.
47+
ASP.NET Core uses git submodules to include the source from a few other projects.
4848
4949
For a new copy of the project, run:
5050
```
@@ -64,19 +64,19 @@ Before opening our .sln files in Visual Studio or VS Code, you need to perform t
6464
```
6565
.\restore.cmd
6666
```
67-
This will download required tools and build the entire repository once. At that point, you should be able to open .sln files to work on the projects you care about.
67+
This will download the required tools and build the entire repository once. At that point, you should be able to open .sln files to work on the projects you care about.
6868
6969
> :bulb: Pro tip: you will also want to run this command after pulling large sets of changes. On the master branch, we regularly update the versions of .NET Core SDK required to build the repo.
7070
> You will need to restart Visual Studio every time we update the .NET Core SDK.
7171
72-
2. Use the `startvs.cmd` script to open Visual Studio .sln files. This script first sets required environment variables.
72+
2. Use the `startvs.cmd` script to open Visual Studio .sln files. This script first sets the required environment variables.
7373
7474
### Solution files
7575
7676
We don't have a single .sln file for all of ASP.NET Core because Visual Studio doesn't currently handle projects of this scale.
7777
Instead, we have many .sln files which include a sub-set of projects. These principles guide how we create and manage .slns:
7878
79-
1. Solution files are not used by CI or command line build scripts. They are for meant for use by developers only.
79+
1. Solution files are not used by CI or command line build scripts. They are meant for use by developers only.
8080
2. Solution files group together projects which are frequently edited at the same time.
8181
3. Can't find a solution that has the projects you care about? Feel free to make a PR to add a new .sln file.
8282
@@ -91,7 +91,7 @@ Opening solution files and building may produce an error code CS0006 with a mess
9191
The cause of this problem is that the solution you are using does not include the project that produces this .dll. This most often occurs after we have added new projects to the repo, but failed to update our .sln files to include the new project. In some cases, it is sometimes the intended behavior of the .sln which has been crafted to only include a subset of projects.
9292
9393
**You can fix this in one of two ways**
94-
1. Build the project on command line. In most cases, running `build.cmd` on command line solve this problem.
94+
1. Build the project on command line. In most cases, running `build.cmd` on command line solves this problem.
9595
2. Update the solution to include the missing project. You can either do this one by one using `dotnet sln`
9696
```
9797
dotnet sln add C:\src\AspNetCore\src\Hosting\Abstractions\src\Microsoft.AspNetCore.Hosting.Abstractions.csproj
@@ -139,7 +139,7 @@ On macOS/Linux:
139139
./build.sh
140140
```
141141

142-
By default, all of the C# projects are built. Some C# projects requires NodeJS to be installed to compile JavaScript assets which are then checked in as source. If NodeJS is detected on the path, the NodeJS projects will be compiled as part of building C# projects. If NodeJS is not detected on the path, the JavaScript assets checked in previously will be used instead. To disable building NodeJS projects, specify /p:BuildNodeJs=false on the command line.
142+
By default, all of the C# projects are built. Some C# projects require NodeJS to be installed to compile JavaScript assets which are then checked in as source. If NodeJS is detected on the path, the NodeJS projects will be compiled as part of building C# projects. If NodeJS is not detected on the path, the JavaScript assets checked in previously will be used instead. To disable building NodeJS projects, specify /p:BuildNodeJs=false on the command line.
143143

144144
### Using `dotnet` on command line in this repo
145145

0 commit comments

Comments
 (0)