Skip to content

fix(dfns): send GITHUB_TOKEN on latest-release query - #392

Merged
wpbonelli merged 1 commit into
MODFLOW-ORG:developfrom
wpbonelli:dfn-sync-token
Oct 13, 2026
Merged

wpbonelli merged 1 commit into
MODFLOW-ORG:developfrom
wpbonelli:dfn-sync-token

Conversation

@wpbonelli

Copy link
Copy Markdown
Member

RemoteDfnRegistry.latest_tag called the GitHub API without a token, so mf dfns sync hit the unauthenticated rate limit on shared CI runners. Send GITHUB_TOKEN when set, like the rest of devtools

RemoteDfnRegistry.latest_tag called the GitHub API without a token, so
`mf dfns sync` hit the unauthenticated rate limit on shared CI runners.
Send GITHUB_TOKEN when set, like the rest of devtools (get_request).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@wpbonelli wpbonelli added this to the 1.10.0 milestone Oct 12, 2026
@wpbonelli wpbonelli added the bug Something isn't working label Oct 12, 2026
@wpbonelli
wpbonelli marked this pull request as ready for review October 12, 2026 23:43
@wpbonelli
wpbonelli merged commit 9d76a59 into MODFLOW-ORG:develop Oct 13, 2026
15 checks passed
@wpbonelli
wpbonelli deleted the dfn-sync-token branch October 13, 2026 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant