Skip to content

Statistics: generate a markdown report from stats.ts - #4294

Open
ddbeck wants to merge 1 commit into
web-platform-dx:mainfrom
ddbeck:statistics-generate-a-markdown-report-from-stats/vtmqyymrlxwp
Open

Statistics: generate a markdown report from stats.ts#4294
ddbeck wants to merge 1 commit into
web-platform-dx:mainfrom
ddbeck:statistics-generate-a-markdown-report-from-stats/vtmqyymrlxwp

Conversation

@ddbeck

@ddbeck ddbeck commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

This generates a Markdown report from stats.ts, for a comment suitable for a successor to #788.

You can see a rendered example in ddbeck#37 (comment).

You can try this yourself by running a command line npm run --silent stats -- --previous-release=latest | npx tsx ./scripts/stats-report.ts -. That said, I don't really expect any humans to run this; it'll be used by GitHub Actions workflows, coming soon in follow up PRs.

Depends on #4293.

Towards #3510.

@github-actions github-actions Bot added the tools and infrastructure Project internal tooling, such as linters, GitHub Actions, or repo settings label Sep 2, 2026
@ddbeck
ddbeck force-pushed the statistics-generate-a-markdown-report-from-stats/vtmqyymrlxwp branch from 637382d to cbe8d0b Compare September 2, 2026 15:49
This is probably only going to ever run in GitHub Actions workflows, so
this doesn't need to be terribly ergonomic, but if you want to try it,
run:

npm run --silent stats -- | npx tsx ./scripts/stats-report.ts -
@ddbeck
ddbeck force-pushed the statistics-generate-a-markdown-report-from-stats/vtmqyymrlxwp branch from cbe8d0b to 53c63aa Compare September 2, 2026 16:00
@ddbeck
ddbeck marked this pull request as ready for review September 2, 2026 16:01
@ddbeck
ddbeck requested a review from Elchi3 September 2, 2026 16:01

@Elchi3 Elchi3 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to work. Two questions:

  • Is "Net" supposed to be a number like "+42"? We don't want percentages anymore?
  • The "Before" and "After" headings have no date anymore, leaving the reader wondering: before and after what?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tools and infrastructure Project internal tooling, such as linters, GitHub Actions, or repo settings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants