Skip to content

fix: make star update metrics truthful and testable - #2

Open
DorianChn wants to merge 1 commit into
mainfrom
codex/fix-trending-window-20260915
Open

DorianChn wants to merge 1 commit into
mainfrom
codex/fix-trending-window-20260915

Conversation

@DorianChn

Copy link
Copy Markdown
Owner

Summary

  • Replace the hard-coded 2025 repository activity window with a moving daily/weekly/monthly window.
  • Stop labeling total GitHub stars as weekly growth; the API does not provide weekly star deltas.
  • Update README wording to describe the data accurately.
  • Add regression tests for the moving window and total-star label, and run them in CI.

Validation

  • python -m py_compile scripts/update_stars.py scripts/test_update_stars.py
  • python scripts/test_update_stars.py -v ✅ 2 tests passed
  • PWA metadata and HTML entrypoint validation ✅
  • git diff --check

This PR is independent of the existing XSS hardening PR #1 and does not modify the generated radar data.

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant