Skip to content

Conversation

@Diaphteiros
Copy link
Contributor

@Diaphteiros Diaphteiros commented Jun 24, 2025

What this PR does / why we need it:
We are seeing problems with rate limiting for installing the task tool.
According to the documentation of the action, authenticating should increase it.

Which issue(s) this PR fixes:
None

Special notes for your reviewer:
Is it a security risk to pass the token into a 3rd-party GHA? I tried to find out what exactly the action does, but for some reason, there is a 12000+ lines javascript file ...

Release note:

Use authentication for the `arduino/setup-task@v2` action to resolve problems with rate limiting.

@Diaphteiros Diaphteiros requested a review from reshnm June 24, 2025 07:27
Copy link
Member

@maximiliantech maximiliantech left a comment

Choose a reason for hiding this comment

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

LGTM

@n3rdc4ptn n3rdc4ptn merged commit 7b89b9e into main Jun 30, 2025
3 checks passed
@n3rdc4ptn n3rdc4ptn deleted the fix-rate-limit branch June 30, 2025 13:37
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.

5 participants