Переглянути джерело

Added minor imporovements to CONTRIBUTING file

Alejandro Celaya 3 роки тому
батько
коміт
035743ef6a
1 змінених файлів з 14 додано та 11 видалено
  1. 14 11
      CONTRIBUTING.md

+ 14 - 11
CONTRIBUTING.md

@@ -1,8 +1,8 @@
 # Contributing
 
-This file will guide you through the process of getting to project up and running, in order to provide coding contributions.
+This file will guide you through the process of getting to project up and running, in case you want to provide coding contributions.
 
-You will also see how to ensure the code fulfills the expected code checks, and how to end up creating a pull request.
+You will also see how to ensure the code fulfills the expected code checks, and how to creating a pull request.
 
 ## System dependencies
 
@@ -23,11 +23,11 @@ Then you will have to follow these steps:
 * Copy the file `docker-compose.override.yml.dist` by also removing the `dist` extension.
 * Start-up the project by running `docker-compose up`.
 
-    The first time this command is run, it will create several containers that are used during development, and may tike some time.
+    The first time this command is run, it will create several containers that are used during development, so it may take some time.
 
     It will also create some empty databases and install the project dependencies with composer.
 
-* Run `./indocker bin/cli db:create` to create an empty database.
+* Run `./indocker bin/cli db:create` to create the initial database.
 * Run `./indocker bin/cli db:migrate` to get database migrations up to date.
 * Run `./indocker bin/cli api-key:generate` to get your first API key generated.
 
@@ -38,21 +38,24 @@ Once you finish this, you will have the project exposed in ports `8080` through
 ## Running code checks
 
 * Run `./indocker composer cs` to check coding styles are fulfilled.
-* Run `./indocker composer cs:fix` to fix coding styles (some may not be fixeable from the CLI)
-* Run `./indocker composer stan` to check the code with phpstan. This tool is the closest you have to "compile" PHP and verify everything would work as expected.
+* Run `./indocker composer cs:fix` to fix coding styles (some may not be fixable from the CLI)
+* Run `./indocker composer stan` to statically analyze the code with [phpstan](https://phpstan.org/). This tool is the closest to "compile" PHP and verify everything would work as expected.
 * Run `./indocker composer test:unit` to run the unit tests.
-* Run `./indocker composer test:db` to run integration tests with the database.
+* Run `./indocker composer test:db` to run the database integration tests.
 
-    This command runs the same test suite against all supported database engines. If you just want to run one of them, you can add one of `:sqlite`, `:mysql`, `:maria`, `:postgres`, `:mssql` to the command to run just one of them.
+    This command runs the same test suite against all supported database engines. If you just want to run one of them, you can add one of `:sqlite`, `:mysql`, `:maria`, `:postgres`, `:mssql` at the end of the command.
     
     For example, `test:db:postgres`.
 
 * Run `./indocker composer test:api` to run API E2E tests. For these, the MySQL database engine is used.
+* Run `./indocker composer infect:test` ti run both unit and database tests (over sqlite) and then apply mutations to them with [infection](https://infection.github.io/).
 
-> Note: Due to some limitations in the tooling used by shlink, the testing databases need to exist first, both for db and api tests (except sqlite).
+> Note: Due to some limitations in the tooling used by shlink, the testing databases need to exist beforehand, both for db and api tests (except sqlite).
 >
-> However, they just need to be created empty, with no tables. Also, once created, they are automatically reset before each execution.
+> However, they just need to be created empty, with no tables. Also, once created, they are automatically reset before every new execution.
 >
-> The testing database is always called `shlink_testing`. You can create it using the database client of your choice. [DBeaver](https://dbeaver.io/) is a good multi-platform desktop database client which supports all the engines supported by shlink.
+> The testing database is always called `shlink_test`. You can create it using the database client of your choice. [DBeaver](https://dbeaver.io/) is a good multi-platform desktop database client which supports all the engines supported by shlink.
 
 ## Pull request process
+
+