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

[feat]: Output GraphQL rate limit #2807

Merged
merged 2 commits into from Nov 13, 2023
Merged

[feat]: Output GraphQL rate limit #2807

merged 2 commits into from Nov 13, 2023

Conversation

SlyckLizzie
Copy link
Contributor

@SlyckLizzie SlyckLizzie commented Nov 5, 2023 •
edited

Resolves #N/A

Before the change?

  • The rate limit client did not output limits for graph ql

After the change?

  • The rate limit client outputs rate limits for graph ql

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

@SlyckLizzie SlyckLizzie changed the title add in graphql rate limit [feat]: Output GraphQL rate limit Nov 5, 2023
@nickfloyd nickfloyd added the Type: Feature New feature or request label Nov 13, 2023
nickfloyd
nickfloyd approved these changes Nov 13, 2023
View reviewed changes
Copy link
Contributor

@nickfloyd nickfloyd left a comment

Choose a reason for hiding this comment

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

Thank you for the commits here @dirtygooback ❤️

@nickfloyd nickfloyd merged commit f679108 into octokit:main Nov 13, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nickfloyd nickfloyd nickfloyd approved these changes

Assignees
No one assigned
Labels
Type: Feature New feature or request
Projects
🧰 Octokit Active
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@SlyckLizzie @nickfloyd