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

fix: linting issues #494

Merged
merged 1 commit into from
Feb 13, 2025
Merged

fix: linting issues #494

merged 1 commit into from
Feb 13, 2025

Conversation

nickfloyd
Copy link
Contributor

This is a blocker for the release to get previous merged work shipped.

@nickfloyd nickfloyd requested a review from wolfy1339 February 13, 2025 17:53
Copy link

👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labeled with Status: Up for grabs. You & others like you are the reason all of this works! So thank you & happy coding! 🚀

@nickfloyd nickfloyd merged commit c346f5c into main Feb 13, 2025
9 checks passed
@nickfloyd nickfloyd deleted the fix-lint branch February 13, 2025 17:58
Copy link

🎉 This PR is included in version 6.1.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

@wolfy1339
Copy link
Member

I backported the changes into v5, though I completely forgot to do a PR and pushed it directly to the branch.

https://github.com/octokit/request-error.js/releases/tag/v5.1.1

Could you update the vulnerability with this patched version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants