Apply suggestions from code review

Co-authored-by: Andy Butland <[email protected]>

Author: nikolajlauridsen

15/umbraco-cms/fundamentals/setup/server-setup/README.md

@@ -32,4 +32,4 @@ The runtime mode setting optimizes Umbraco for the best development experience o
 
 ## [Running Umbraco in Docker](running-umbraco-in-docker.md)
 
-Overview of consideration when running Umbraco in Docker.
+Overview of topics to consider when running Umbraco in Docker.

15/umbraco-cms/fundamentals/setup/server-setup/running-umbraco-in-docker.md

@@ -1,25 +1,31 @@
 # Running Umbraco in Docker
 
-Exactly how you choose to compose your Dockerfile depends on your needs, and your project. This section is not intended as a guide, 
-but as a general overview of what to be aware of when hosting in Docker.
+Exactly how you choose to compose your Dockerfile will depend on your project specific needs. This section is not intended as a comprehensive guide, rather as an overview of topics to be aware of when hosting in Docker.
 
 ## What is Docker
 
-Docker is a platform for developing, shipping, and running applications in containers. There exist multiple services for hosting these containers, 
-for more information, [refer to the official Docker documentation](https://docs.docker.com/)
+Docker is a platform for developing, shipping, and running applications in containers. Multiple services exist for hosting these containers.  For more information, [refer to the official Docker documentation](https://docs.docker.com/)
 
 ## The Docker file system
 
-By default, all files created inside a container are written to an ephemeral writable container layer. 
+By default, files created inside a container are written to an ephemeral, writable container layer. 
 This means that the files don't persist when the container is removed, and it's challenging to get files out of the container. Additionally, this writable layer is not suitable for performance-critical data processing.
-This has implications when running Umbraco in Docker. For more information, refer to the [Docker documentation on storage](https://docs.docker.com/engine/storage/).
+
+This has implications when running Umbraco in Docker.
+
+For more information, refer to the [Docker documentation on storage](https://docs.docker.com/engine/storage/).
 
 ### General file system consideration 
 
-In general, when working with files and Docker you work in a "push" fashion with the read-only layers. WWhen you build, you take all your files and "push" them into the read-only layer.
-This means that you should avoid making files on the fly, and instead rely on building your image. You should not create or edit template files on the fly, the same goes for script and style files. 
+In general, when working with files and Docker you work in a "push" fashion with read-only layers. When you build, you take all your files and "push" them into this read-only layer.
+
+This means that you should avoid making files on the fly, and instead rely on building your image.
+
+In an Umbraco context, this means you should not create or edit template, script or stylesheet files via the backoffice. These should be deployed as part of your web application and not managed via Umbraco. 
+
+Similarly, you shouldn't use InMemory modelsbuilder, since that also relies on creating files on the disk. While this is not a hard requirement, it doesn't provide any value if not live editing your site.
 
-Similarly, you shouldn't use InMemory modelsbuilder, since that also relies on creating files on the disk. While this is not a hard requirement, it doesn't provide any value if not live editing your site. Instead, use source code in development, and none in production, as [described when using runtime modes](https://docs.umbraco.com/umbraco-cms/fundamentals/setup/server-setup/runtime-modes).
+Instead, configure models builder to use use "source code" mode in development, and "none" in production, as [described when using runtime modes](https://docs.umbraco.com/umbraco-cms/fundamentals/setup/server-setup/runtime-modes).
 
 
 ### Logs
@@ -33,20 +39,19 @@ The `/umbraco/Data/` directory is used to store temporary files, such as file up
 
 ### Media
 
-Similarly to logs, it's recommended to not store media in the writable layer, both for performance reasons, 
-but also for practical development reasons. You likely want to persist media files between containers. 
+It's recommended to not store media in the writable layer. This is for similar performance reasons as logs, 
+but also for practical hosting reasons. You likely want to persist media files between containers. 
 
-One possible solution here is to again use bind mounts. The ideal solution is store the media and ImageSharp cache externally, 
-for more information on this, refer to the [Azure Blob Storage documentation](https://docs.umbraco.com/umbraco-cms/extending/filesystemproviders/azure-blob-storage).
+One possible solution here is to again use bind mounts. The ideal setup though is to store the media and ImageSharp cache externally. For more information on this, refer to the [Azure Blob Storage documentation](https://docs.umbraco.com/umbraco-cms/extending/filesystemproviders/azure-blob-storage).
 
 ### Required files
 
-If your solution requires some files to run, for instance license files. You need to pass these files into the container at build time, or mount them externally. 
+Your solution may require some specific files to run, such as license files. You will need to pass these files into the container at build time, or mount them externally. 
 
 ## HTTPS
 
-When running in websites in Docker, it's common to use do so behind a reverse proxy, or load balancers.
-In these scenarios you're likely to handle SSL termination at the reverse proxy. This means that Umbraco will not be aware of the SSL termination, and will likely complain about not using HTTPS.
+When running in websites in Docker, it's common to use do so behind a reverse proxy, or load balancer.
+In these scenarios you will likely handle SSL termination at the reverse proxy. This means that Umbraco will not be aware of the SSL termination, and will complain about not using HTTPS.
 
 Umbraco checks for HTTPS in two locations:
 
@@ -57,7 +62,7 @@ To avoid these checks failing, you can remove them in your project.
 
 ### Health Check
 
-The health check must be removed via configuration, either through the `appsettings.json`, environment variables, or similar, for more information see the [Health Check documentation](../../../reference/configuration/healthchecks.md).
+The health check must be removed via configuration, through the `appsettings.json` file, environment variables, or similar. For more information see the [Health Check documentation](../../../reference/configuration/healthchecks.md).
 
 The `HstsCheck` key is `E2048C48-21C5-4BE1-A80B-8062162DF124` so the appsettings will look something like:
 
@@ -76,7 +81,7 @@ The `HstsCheck` key is `E2048C48-21C5-4BE1-A80B-8062162DF124` so the appsettings
 
 ### Runtime mode validator
 
-The `UseHttpsValidator` must be removed through code, for more information see the [Runtime mode documentation](runtime-modes.md).
+The `UseHttpsValidator` must be removed through code For more information see the [Runtime mode documentation](runtime-modes.md).
 
 The code to remove the validator can look something like: