Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ci: avoid ratelimits by authenticating ourselves, fixes #419 #420

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

ci: avoid ratelimits by authenticating ourselves, fixes #419 #420

wants to merge 1 commit into from

Conversation

klaernie
Copy link
Member

This is a fairly simple change, but I think solves our problem from #419

@klaernie klaernie requested a review from theoludwig January 13, 2025 21:10
theoludwig
theoludwig approved these changes Jan 14, 2025
View reviewed changes
Copy link
Member

@theoludwig theoludwig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed, it seems to resolve the issue, as the CI work first try.

However, I would not call this commit a "fix", as we don't need to create a release for it.

@theoludwig theoludwig changed the title fix: help the CI avoid ratelimits by authenticating ourselves, fixes #419 ci: avoid ratelimits by authenticating ourselves, fixes #419 Jan 14, 2025
@theoludwig theoludwig linked an issue Jan 14, 2025 that may be closed by this pull request
[Improvement] Unreliable CI because of GitHub Rate Limiting #419
Open
@klaernie
Copy link
Member Author

However, I would not call this commit a "fix", as we don't need to create a release for it.

Good point. I should put a cheat sheet with the conventional commit prefixes somewhere ;)

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

@theoludwig theoludwig theoludwig approved these changes

@mstruebing mstruebing mstruebing approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Improvement] Unreliable CI because of GitHub Rate Limiting
3 participants
@klaernie @mstruebing @theoludwig