Skip to content

src/git: define "onSecondaryRateLimit" callback#154

Open
yosiat wants to merge 2 commits into
Redocly:mainfrom
yosiat:fix-secondary-rate-limit-err
Open

src/git: define "onSecondaryRateLimit" callback#154
yosiat wants to merge 2 commits into
Redocly:mainfrom
yosiat:fix-secondary-rate-limit-err

Conversation

@yosiat

@yosiat yosiat commented Feb 1, 2026

Copy link
Copy Markdown

What/Why/How?

@octokit/plugin-throttling requires onSecondaryRateLimit to be defined. will resolve #139

Reference

Testing

Tested on my internal pointing at my fork.

Screenshots (optional)

Check yourself

  • Code is linted
  • Tested
  • All new/updated code is covered with tests

Security

  • Security impact of change has been considered
  • Code follows company security practices and guidelines

`@octokit/plugin-throttling` requires `onSecondaryRateLimit` to be defined.
will resolve Redocly#139
Comment thread dist/index.js
@@ -14788,6 +14788,22 @@ function charFromCodepoint(c) {
);

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Please let me know if I should push dist changes in my PR, or there is CI/CD pipeline which auto handles it for me.

@yosiat yosiat marked this pull request as ready for review February 1, 2026 11:58
@yosiat

yosiat commented Mar 24, 2026

Copy link
Copy Markdown
Author

Hey @roman-sainchuk / @SmoliyY (pinging you, see I since you are the latest committers).. is it possible to look at this PR?

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.

octokit error on onSecondaryRateLimit

1 participant