fix: collect PyPI metrics weekly alongside GitHub metrics#73
Merged
Conversation
- Add `requests` to pyproject.toml dependencies (was missing, causing silent failures in collect_pypi_metrics.py) - Switch `collect-metrics.yml` from `uv pip install --system pyyaml` to `uv sync` to use the lock file properly - Use `uv run python` consistently for all script invocations - Add PyPI metrics collection step to the weekly Monday schedule, committing both github and pypi metric files together PyPI data was last updated 2025-12-03 due to `requests` not being installed during the collect-metrics workflow run. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
requestswas missing frompyproject.toml, causingcollect_pypi_metrics.pyto silently fail (it hascontinue-on-error: true) — PyPI data has been stale since 2025-12-03collect-metrics.ymlwas usinguv pip install --system pyyamlinstead ofuv sync, so the lock file wasn't being useddeploy-catalog-gcp.yml, not on a dedicated weekly scheduleChanges
requests>=2.32.0topyproject.tomlcore dependencies and updateuv.lockcollect-metrics.ymlto useuv syncfor dependency installationuv run pythonconsistently for all script invocations in the workflowcollect-metrics.ymlworkflow, staging both GitHub and PyPI JSON files in the same commitTest plan
collect-metrics.ymlworkflow run succeeds and commits updatedpypi_metrics.jsonwith a fresh timestamp