We should write an ADR (Architectural Decision Record) documenting the outcome of T368336. This ADR doesn't have to be long or complex, but it should list which options we considered, state which one we chose, and explain why. An example of a brief ADR that's similar in nature is here.
This ADR could be a wiki page on mediawiki.org, or a Markdown file in the extension repo.
| Subject | Author | Repo | Branch | Lines +/- | |
|---|---|---|---|---|---|
| Fixes to markdown syntax | Jdlrobson | mediawiki/extensions/Chart | master | +8 -3 | |
| Documents chart library selection | Jdlrobson | mediawiki/extensions/Chart | master | +75 -0 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T374065 Document all the technicals decisions we've made so far in the codebase | |||
| Resolved | Jdlrobson | T368338 Document chart library selection | |||
| Resolved | Catrope | T368336 Select a chart library |
Change #1071977 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):
[mediawiki/extensions/Chart@master] Documents chart library selection
Change #1071977 merged by jenkins-bot:
[mediawiki/extensions/Chart@master] Documents chart library selection
@NBaca-WMF curious if you are keen to sign off ? https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/extensions/Chart/+/refs/heads/master/doc/adr
@Jdlrobson looks like there are some formatting issues with the markdown but apart from that looks good to go.
Change #1073288 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):
[mediawiki/extensions/Chart@master] Fixes to markdown syntax
Change #1073288 merged by jenkins-bot:
[mediawiki/extensions/Chart@master] Fixes to markdown syntax