From 64a0aef48bad291deab23c0a8147e051c3c657c0 Mon Sep 17 00:00:00 2001 From: Arthur Lu Date: Sun, 13 Nov 2022 05:25:56 +0000 Subject: [PATCH] update pipeline arch linting and unit testing ADRs --- specs/adrs/111222-cicd-multiple-pipelines.md | 19 +++++++++++++++++++ specs/adrs/111222-jslinting-eslint.md | 19 +++++++++++++++++++ specs/adrs/111222-jsunittest-mocha.md | 19 +++++++++++++++++++ 3 files changed, 57 insertions(+) create mode 100644 specs/adrs/111222-cicd-multiple-pipelines.md create mode 100644 specs/adrs/111222-jslinting-eslint.md create mode 100644 specs/adrs/111222-jsunittest-mocha.md diff --git a/specs/adrs/111222-cicd-multiple-pipelines.md b/specs/adrs/111222-cicd-multiple-pipelines.md new file mode 100644 index 0000000..62495a2 --- /dev/null +++ b/specs/adrs/111222-cicd-multiple-pipelines.md @@ -0,0 +1,19 @@ +# Use multiple CI/CD pipelines in parallel + +- Status: accept +- Deciders: Arthur Lu, Marc Reta +- Date: 11 / 12 / 22 + +## Decision Drivers + +- Need to perform many different CI/CD tasks +- Need pipeline to be durable against any single failure + +## Considered Options + +- Single deep pipeline +- Multiple short pipelines in parallel + +## Decision Outcone + +Chosen Option: Multiple short pipelines in parallel as any one failure will not prevent code from being developed and deployed. \ No newline at end of file diff --git a/specs/adrs/111222-jslinting-eslint.md b/specs/adrs/111222-jslinting-eslint.md new file mode 100644 index 0000000..1b3b972 --- /dev/null +++ b/specs/adrs/111222-jslinting-eslint.md @@ -0,0 +1,19 @@ +# Use eslint for JS linting framework + +- Status: accept +- Deciders: Arthur Lu, Marc Reta +- Date: 11 / 12 / 22 + +## Decision Drivers + +- Need linting to work with multiple style standards +- Need linting to be fast and informative + +## Considered Options + +- JSLint +- eslint + +## Decision Outcome + +Chosen Option: eslint for its low configuration complexity. \ No newline at end of file diff --git a/specs/adrs/111222-jsunittest-mocha.md b/specs/adrs/111222-jsunittest-mocha.md new file mode 100644 index 0000000..f128c0e --- /dev/null +++ b/specs/adrs/111222-jsunittest-mocha.md @@ -0,0 +1,19 @@ +# Use mocha for JS unit testing framework + +- Status: accept +- Deciders: Arthur Lu, Marc Reta +- Date: 11 / 12 / 22 + +## Decision Drivers + +- Need specification on how to write unit testing assertion statements +- Need framework to perform unit testing quickly for immediate code feedback + +## Considered Options +- JUnit5 +- Jest +- Mocha + +## Decision Outcome + +Chosen Option: Mocha because it was significantly faster to run than Jest (1s vs 30s), and can use a variety of assertion styles unlike JUnit5. \ No newline at end of file