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.

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

Update rate limit feature to be more browser freindly #3428

Merged
merged 9 commits into from
Jan 17, 2024
Merged

Conversation

jdalrymple
Copy link
Owner

@jdalrymple jdalrymple commented Oct 11, 2023

fixes #3427
fixes #3493

  • Switch micromatch to picomatch-browser
  • Awaiting PR from async-sema
  • Use rate-limiter-flexible instead of async-sema for rate limiting
📦 Published PR as canary version: 39.30.0--canary.3428.1137971077.0

✨ Test out this PR locally via:

npm install @gitbeaker/[email protected]
npm install @gitbeaker/[email protected]
npm install @gitbeaker/[email protected]
npm install @gitbeaker/[email protected]
# or 
yarn add @gitbeaker/[email protected]
yarn add @gitbeaker/[email protected]
yarn add @gitbeaker/[email protected]
yarn add @gitbeaker/[email protected]

@jdalrymple jdalrymple added the type:bug Changes fix a minor bug label Oct 11, 2023
@jdalrymple
Copy link
Owner Author

Just waiting for picomatch to update instead of using a diff dep. They seem ready to update soon though!

@jdalrymple jdalrymple merged commit 18d4123 into main Jan 17, 2024
6 checks passed
@jdalrymple jdalrymple deleted the bug/3427 branch January 17, 2024 19:49
@jdalrymple jdalrymple removed the release:canary Deploy canary release label Jan 17, 2024
@jdalrymple
Copy link
Owner Author

🚀 PR was released in 39.30.0 🚀

@jdalrymple jdalrymple added the released This issue/pull request has been released. label Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released This issue/pull request has been released. type:bug Changes fix a minor bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

gitbreaker prevents the node.js process from exiting SPA compatibility is gone since v39.19.0
1 participant