Skip to content

Commit

Permalink
Status Chart: Remove videos, hide bugs/issues by default (microsoft#4235
Browse files Browse the repository at this point in the history
)
  • Loading branch information
StephanTLavavej authored Dec 5, 2023
1 parent ddc5d67 commit 123a45d
Show file tree
Hide file tree
Showing 11 changed files with 1,718 additions and 1,825 deletions.
1 change: 0 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@

# Ignore the TypeScript compiler's output.
/built/gather_stats.js
/built/video_table.js

# Ignore installed packages.
/node_modules/
Expand Down
4 changes: 1 addition & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ it'll update `package.json` and `package-lock.json` accordingly. `git add` and `
`npm install [package1]@latest [package2]@latest [...]` for all of the packages listed
as `dependencies` in `package.json` (e.g. `@octokit/graphql@latest dotenv@latest`).
+ Ensure that the `importmap` in `index.html` remains synchronized to the same versions.
* Update other dependencies in `index.html` (e.g. Primer CSS and es-module-shims).
* Update other dependencies in `index.html` (e.g. Primer CSS).
* Update `weekly_table.ts` by adding a new row.
+ We update it every Friday, although nothing bad will happen if we skip a week or update it on a different day.
+ `vso` is the number of Active work items under the STL's Area Path.
Expand All @@ -85,7 +85,6 @@ it'll update `package.json` and `package-lock.json` accordingly. `git add` and `
```
python tools/count_libcxx.py STL_REPO/tests/libcxx/expected_results.txt
```
* Update `video_table.ts` if any [code review videos][] have been recorded or uploaded.
* Run `npm run gather` to compile `gather_stats.ts` and then regenerate `daily_table.ts` and `monthly_table.ts`.
+ This regenerates the files from scratch, but the diff should be small because the data is stable and the process
is deterministic.
Expand Down Expand Up @@ -113,7 +112,6 @@ Copyright (c) Microsoft Corporation.
SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
[Chart.js]: https://www.chartjs.org/
[code review videos]: https://github.com/microsoft/STL/wiki/Code-Review-Videos
[Node.js]: https://nodejs.org/en/
[Personal Access Tokens]: https://github.com/settings/tokens
[Status Chart]: https://microsoft.github.io/STL/
Expand Down
2 changes: 1 addition & 1 deletion built/status_chart.mjs

Large diffs are not rendered by default.

10 changes: 3 additions & 7 deletions index.html
Original file line number Diff line number Diff line change
Expand Up @@ -7,13 +7,13 @@
<!-- SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception -->
<meta name="viewport" content="width=device-width, initial-scale=1">
<title>STL Status Chart</title>
<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/@primer/css@21.0.9/dist/primer.min.css" />
<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/@primer/css@21.1.0/dist/primer.min.css" />
<script type="importmap">
{ "imports": {
"@kurkle/color": "https://cdn.jsdelivr.net/npm/@kurkle/color@0.3.2/dist/color.esm.min.js",
"chart.js": "https://cdn.jsdelivr.net/npm/chart.js@4.4.0/dist/chart.min.js",
"chart.js": "https://cdn.jsdelivr.net/npm/chart.js@4.4.1/dist/chart.min.js",
"chartjs-adapter-luxon": "https://cdn.jsdelivr.net/npm/chartjs-adapter-luxon@1.3.1/dist/chartjs-adapter-luxon.esm.min.js",
"luxon": "https://cdn.jsdelivr.net/npm/luxon@3.4.3/build/es6/luxon.min.js"
"luxon": "https://cdn.jsdelivr.net/npm/luxon@3.4.4/build/es6/luxon.min.js"
} }
</script>
<script type="module" src="built/status_chart.mjs"></script>
Expand Down Expand Up @@ -66,10 +66,6 @@ <h2>Explanation</h2>
id="currentValue-pr"></span>
<a href="https://github.com/microsoft/STL/pulls">Open pull requests.</a>
</li>
<li><span class="Counter color-bg-accent-emphasis color-fg-on-emphasis" id="currentValue-video"></span>
<a href="https://github.com/microsoft/STL/wiki/Code-Review-Videos">Code review videos</a>
that have been recorded but not yet transcribed and uploaded.
</li>
</ul>
</li>
<li>Left axis:
Expand Down
Loading

0 comments on commit 123a45d

Please sign in to comment.