|
7 | 7 | - Use severities: **[Blocker]**, **[Important]**, **[Suggestion]**. |
8 | 8 | - Do **not** nitpick items handled by automation (formatting, lint rules). |
9 | 9 |
|
10 | | -Related docs: |
| 10 | +Related docs (important): |
11 | 11 |
|
12 | | -- **Coding Standards:** https://github.com/wireapp/wire-webapp/tree/docs/coding-standards.md |
13 | | -- **Tech Radar:** https://github.com/wireapp/wire-webapp/tree/docs/tech-radar.md |
| 12 | +- **Coding Standards:** docs/coding-standards.md |
| 13 | +- **Tech Radar:** docs/tech-radar.md |
| 14 | + |
| 15 | +**Additional rules (open when relevant)** |
| 16 | + |
| 17 | +- Security: `.github/instructions/security.instructions.md` |
| 18 | +- Accessibility: `.github/instructions/accessibility.instructions.md` |
| 19 | +- React/UX: `.github/instructions/react.instructions.md` |
| 20 | +- TypeScript: `.github/instructions/typescript.instructions.md` |
14 | 21 |
|
15 | 22 | --- |
16 | 23 |
|
@@ -50,12 +57,12 @@ Related docs: |
50 | 57 |
|
51 | 58 | ### Everything else |
52 | 59 |
|
53 | | -- For imports/TS/React/testing/naming/readability: **refer to** the [Coding Standards](https://github.com/wireapp/wire-webapp/tree/docs/coding-standards.md). |
| 60 | +- For imports/TS/React/testing/naming/readability: **refer to** the [Coding Standards](docs/coding-standards.md). |
54 | 61 | - If a standard is violated, link the relevant section and suggest a minimal change. |
55 | 62 |
|
56 | 63 | ### Technology choices |
57 | 64 |
|
58 | | -- Compare any new dependencies in `package.json`/lockfiles to the [Tech Radar](https://github.com/wireapp/wire-webapp/tree/docs/tech-radar.md). |
| 65 | +- Compare any new dependencies in `package.json`/lockfiles to the [Tech Radar](docs/tech-radar.md). |
59 | 66 | - If not **Adopt**/**Trial**, mark **[Blocker]** and request an RFC/approval link. |
60 | 67 | - For **Trial**, ensure usage is narrowly scoped and success criteria exist. |
61 | 68 |
|
|
0 commit comments