Skip to content

[patch] Updated readme.py to use pypistats instead of pepy #63

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 2, 2025

Conversation

michaelthomasletts
Copy link
Owner

@michaelthomasletts michaelthomasletts commented Jul 2, 2025

All submissions

  • Did you include the version part parameter, i.e. [major | minor | patch], to the beginning of the pull request title so that the version is bumped correctly?
    • Example pull request title: '[minor] Added a new parameter to the RefreshableSession object.'
    • Note: the version part parameter is only required for major and minor updates. Patches may exclude the part parameter from the pull request title, as the default is 'patch'.
  • Did you verify that your changes pass pre-commit checks before opening this pull request?
    • The pre-commit checks are identical to required status checks for pull requests in this repository. Know that suppressing pre-commit checks via the --no-verify | -nv arguments will not help you avoid the status checks!
    • To ensure that pre-commit checks work on your branch before running git commit, run pre-commit install and pre-commit install-hooks beforehand.
  • Have you checked that your changes don't relate to other open pull requests?

New feature submissions

  • Does your new feature include documentation? If not, why not?
    • Does that documentation match the numpydoc guidelines?
    • Did you locally test your documentation changes using sphinx-build doc doc/_build from the root directory?
  • Did you write unit tests for the new feature? If not, why not?
    • Did the unit tests pass?
    • Did you know that locally running unit tests requires an AWS account?
      • You must create a ROLE_ARN environment variable on your machine using export ROLE_ARN=<your role arn here>.

Submission details

Pepy has been returning a 401 error lately. In order to continue displaying total downloads statistics, I've decided to go with pypistats instead.

@michaelthomasletts michaelthomasletts merged commit 6018638 into main Jul 2, 2025
2 checks passed
@michaelthomasletts michaelthomasletts deleted the rm/readme branch July 2, 2025 05:50
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