Skip to content

memory_report.py: suppress the detailed table if all changes are <0.01MB big#23212

Merged
AlexWaygood merged 2 commits into
mainfrom
alex/ci-script-improvements
Feb 11, 2026
Merged

memory_report.py: suppress the detailed table if all changes are <0.01MB big#23212
AlexWaygood merged 2 commits into
mainfrom
alex/ci-script-improvements

Conversation

@AlexWaygood

Copy link
Copy Markdown
Member

No description provided.

@AlexWaygood AlexWaygood added ci Related to internal CI tooling ty Multi-file analysis & type inference labels Feb 10, 2026
@astral-sh-bot

astral-sh-bot Bot commented Feb 10, 2026

Copy link
Copy Markdown

Memory usage report

Memory usage unchanged ✅

@astral-sh-bot

astral-sh-bot Bot commented Feb 10, 2026

Copy link
Copy Markdown

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

@ibraheemdev

Copy link
Copy Markdown
Member

#23216 changes the report to print memory in bytes/kilobytes if the size is small. I think we probably instead want to suppress any diff row (whether the entire project or a given item) if the percentage diff rounds to 0.0%?

Comment thread scripts/memory_report.py Outdated
@AlexWaygood AlexWaygood marked this pull request as draft February 10, 2026 22:59
@AlexWaygood AlexWaygood force-pushed the alex/ci-script-improvements branch 2 times, most recently from fa63436 to 14fcf1c Compare February 11, 2026 14:02
@AlexWaygood AlexWaygood marked this pull request as ready for review February 11, 2026 14:08
@AlexWaygood AlexWaygood force-pushed the alex/ci-script-improvements branch from 14fcf1c to f1faadb Compare February 11, 2026 14:10
@AlexWaygood AlexWaygood merged commit 40fd2f1 into main Feb 11, 2026
43 checks passed
@AlexWaygood AlexWaygood deleted the alex/ci-script-improvements branch February 11, 2026 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Related to internal CI tooling ty Multi-file analysis & type inference

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants