diff --git a/content/Journey/_index.md b/content/Journey/_index.md index 2c3fff2..940f810 100644 --- a/content/Journey/_index.md +++ b/content/Journey/_index.md @@ -56,7 +56,7 @@ Having more than one reviewer on a PR is a process smell for issues with the qua ### Tests are deferred or skipped -We must treat tests as first-class citizens. They are more important than the code we are testing. They cannot be an afterthought and we need to make sure we are using them to increase our confidence rather than meeting some arbitrary ["coverage metric"](https://dojoconsortium.org/metrics/code-coverage/). We don't need all of the legacy code to be 100% tested, or even 1% tested to start CI. We need to commit to "we will never push untested changes." Timelines are irrelevant if we deliver things on time that are broken. +We must treat tests as first-class citizens. They are more important than the code we are testing. They cannot be an afterthought and we need to make sure we are using them to increase our confidence rather than meeting some arbitrary ["coverage metric"](https://dojoconsortium.org/docs/metrics/code-coverage/). We don't need all of the legacy code to be 100% tested, or even 1% tested to start CI. We need to commit to "we will never push untested changes." Timelines are irrelevant if we deliver things on time that are broken. ### The team lacks knowledge on how to write tests for CI diff --git a/content/MinimumCD/CI/_index.md b/content/MinimumCD/CI/_index.md index 989f222..b2a1735 100644 --- a/content/MinimumCD/CI/_index.md +++ b/content/MinimumCD/CI/_index.md @@ -62,4 +62,4 @@ You don't need any tests in existing code to begin. You need to test new code wi ### What code coverage percentage should we set as a standard for all teams? -We shouldn't. Code coverage mandates incentivize meaningless tests that hide the fact that code is not tested. It is better to have no tests than to have tests you do not trust. See the [Dojo Consortium's info](https://dojoconsortium.org/metrics/code-coverage/) on this metric. +We shouldn't. Code coverage mandates incentivize meaningless tests that hide the fact that code is not tested. It is better to have no tests than to have tests you do not trust. See the [Dojo Consortium's info](https://dojoconsortium.org/docs/metrics/code-coverage/) on this metric. diff --git a/data/names/contributors.yml b/data/names/contributors.yml index 953ab74..d8c892f 100644 --- a/data/names/contributors.yml +++ b/data/names/contributors.yml @@ -16,7 +16,7 @@ list: - name: Istvan Bathazi contact: https://www.linkedin.com/in/istvan-bathazi - name: Kaine Bent - contact: https://www.ka1ne.com + contact: https://www.linkedin.com/in/ka1ne/ - name: Marc Boudreau contact: https://www.linkedin.com/in/marc-boudreau - name: Kelly Brownsberger diff --git a/data/names/signatures.yml b/data/names/signatures.yml index 3105050..e592aa8 100644 --- a/data/names/signatures.yml +++ b/data/names/signatures.yml @@ -373,3 +373,5 @@ list: contact: https://www.linkedin.com/in/jarkko-piiroinen/ - name: Marc Loupias contact: https://www.linkedin.com/in/marc-loupias-4aa71491/ + - name: Paul Hammond + contact: https://www.linkedin.com/in/paul-hammond-bb5b78251/