Skip to content

Conversation

@choldgraf
Copy link
Member

This removes a hard-coded list of bot names, and instead uses the GraphQL BOT metadata mentioned in #145.

@choldgraf
Copy link
Member Author

omg why is everything so hard, I don't know why the access token is empty...

Use GITHUB_TOKEN when TOKEN_READONLY secret isn't available (PRs from forks).
This allows tests to run on all PRs while still using the higher-limit PAT
when available on pushed branches.
Fixes pandas UserWarning about creating columns via attribute names.
Uses the proper pandas API for storing metadata on DataFrames.
@choldgraf
Copy link
Member Author

ok well I finally got it working by removing the secret - I forgot it's not available in PRs, but then immediately hit the rate limit.

So I think main is no longer broken if we merge this, but the test suite is still rate limited.

@stefanv stefanv merged commit 7e3a545 into executablebooks:main Dec 4, 2025
2 of 4 checks passed
@stefanv
Copy link
Collaborator

stefanv commented Dec 4, 2025

Nice, thanks!

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.

Use GraphQL API to differentiate bot vs. human PR authors instead of hard-coding names

2 participants