Skip to content

Add PyPI version update notification#41

Merged
anandhu-eng merged 6 commits into
mainfrom
feat/version-check
Jun 18, 2026
Merged

Add PyPI version update notification#41
anandhu-eng merged 6 commits into
mainfrom
feat/version-check

Conversation

@arav-agarwal2

@arav-agarwal2 arav-agarwal2 commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Notifies users when a newer version is available on PyPI. Uses a background thread + 24h file cache so the check never blocks the CLI.

Should resolve #39.

Notifies users when a newer version is available on PyPI. Uses a background
thread + 24h file cache so the check never blocks the CLI.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@arav-agarwal2 arav-agarwal2 marked this pull request as ready for review June 18, 2026 16:27
@github-actions

github-actions Bot commented Jun 18, 2026

Copy link
Copy Markdown

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

arav-agarwal2 and others added 5 commits June 18, 2026 13:05
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Return str() cast in _cached_latest to satisfy no-any-return
- Add type: ignore[assignment] for optional _register_upgrade_notice
- Emit accuracy-file as ERROR (not WARNING) when results.json missing
- Reformat builder.py and test_builder.py per ruff

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add a module-level _registered flag so that calling register_upgrade_notice()
more than once in the same process (e.g. when both CLI entry points are loaded)
only registers a single atexit handler, preventing duplicate upgrade notices.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Fix register_upgrade_notice() to be a true no-op on subsequent calls
@anandhu-eng anandhu-eng merged commit 2f1e024 into main Jun 18, 2026
7 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add pre-emptive version update noticing for users of CLI.

2 participants