Skip to content

Conversation

@chrisopperwall-qz
Copy link
Contributor

@chrisopperwall-qz chrisopperwall-qz commented Oct 22, 2025

What changes this PR introduce?

This explicitly passes a process.env.GITHUB_TOKEN to new Octokit() so that requests via CI can be authenticated and avoid rate limiting.

I tried seeing if there was a way to get around this via octokit, but it seems like the library requires library clients to specify the authentication method explicitly due to the many different ways that the GitHub API can be authenticated with. https://github.com/octokit/authentication-strategies.js/?tab=readme-ov-file#official-authentication-strategies

List any relevant issue numbers

#425 (comment)

Is there anything you'd like reviewers to focus on?

@theoludwig theoludwig changed the title Use GITHUB_TOKEN env if available fix: use GITHUB_TOKEN env if available Oct 22, 2025
Copy link
Member

@theoludwig theoludwig left a comment

Choose a reason for hiding this comment

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

Thanks!

@theoludwig theoludwig merged commit 57ab949 into editorconfig-checker:master Oct 24, 2025
5 checks passed
@github-actions
Copy link

🎉 This PR is included in version 6.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

3 participants