Skip to content

rate-limit: add PAT input to increase the rate limit #3

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
Merged
merged 9 commits into from
May 1, 2025
Merged

rate-limit: add PAT input to increase the rate limit #3

merged 9 commits into from
May 1, 2025

Conversation

@mgovers
Copy link
Member

@mgovers mgovers commented May 1, 2025 •
edited
Loading

Before merge (do-not-merge label):

@mgovers mgovers added do-not-merge This should not be merged feature New feature or request labels May 1, 2025
@mgovers mgovers self-assigned this May 1, 2025
@mgovers mgovers force-pushed the feature/fix-rate-limit branch from 6432b03 to 96e8b1b Compare May 1, 2025 06:08
TonyXiang8787
TonyXiang8787 previously approved these changes May 1, 2025
View reviewed changes
mgovers added 2 commits May 1, 2025 08:11
@mgovers
add PAT, fix rate limit
ee0a631 
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers
actually test the current version of the action + add tokens
ec5a04f 
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers mgovers force-pushed the feature/fix-rate-limit branch from 96e8b1b to ec5a04f Compare May 1, 2025 06:13
mgovers and others added 7 commits May 1, 2025 08:51
@mgovers
Merge branch 'feature/add-test-workflow' into feature/fix-rate-limit
71c787e 
Signed-off-by: Martijn Govers <martijn.govers@alliander.com>
@mgovers
version
f4e6184 
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers
mask is illegal code
9f03292 
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers
fix typo
49af1a9 
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers
apparently, required inputs are not required...
a581701 
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers
use github actions token
5720e37 
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers
Merge branch 'feature/add-test-workflow' into feature/fix-rate-limit
5543a63
Base automatically changed from feature/add-test-workflow to main May 1, 2025 07:46
@figueroa1395 figueroa1395 dismissed TonyXiang8787’s stale review May 1, 2025 07:46

The base branch was changed.

@mgovers mgovers removed the do-not-merge This should not be merged label May 1, 2025
@mgovers mgovers merged commit 7401b7f into main May 1, 2025
5 of 6 checks passed
@mgovers mgovers deleted the feature/fix-rate-limit branch May 1, 2025 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@figueroa1395 figueroa1395 figueroa1395 approved these changes

@TonyXiang8787 TonyXiang8787 TonyXiang8787 left review comments

Assignees

@mgovers mgovers

Labels

feature New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@mgovers @TonyXiang8787 @figueroa1395