Skip to content

Conversation

@choldgraf
Copy link
Member

@choldgraf choldgraf commented Sep 29, 2025

This PR updates our github workflows so that we have higher rate limits without exposing the repository to secrets vulnerabilities. Here's what I've done:

  • Created a PAT that is read-only for public repositories and added it in a "github environment' called pr-tests
  • Moved our PyPI publishing PAT to another environment called pypi-publish.
  • Updated our tests workflow to run pull_request_target so that the tests PAT is available
  • Use that PAT so we have a higher rate limit and don't hit it as easily.
  • Adds some documentation about this.

I'll make a release soon to see if this fixes things.

@choldgraf
Copy link
Member Author

closing in favor of #135

@choldgraf choldgraf closed this Sep 29, 2025
@choldgraf choldgraf reopened this Sep 29, 2025
@choldgraf
Copy link
Member Author

I'm gonna try merging this to see if this fixes the tests and rate limits!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants