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

add github_access_token option #152

Merged
merged 1 commit into from Dec 30, 2022

Conversation

zimbatm
Copy link
Contributor

@zimbatm zimbatm commented Dec 27, 2022

I'm starting to see rate limit issues with GitHub Actions more
frequently so I propose to make this option more readily available.

@zimbatm
Copy link
Contributor Author

zimbatm commented Dec 27, 2022

before I go and fix the CI issues, what do you think, is it worth the extra complexity?

@domenkozar
Copy link
Member

Happy to merge this if CI passes :)

@zimbatm zimbatm force-pushed the github-access-token branch 2 times, most recently from 1286288 to b588e45 Compare December 29, 2022 22:24
I'm starting to see rate limit issues with GitHub Actions more
frequently so I propose to make this option more readily available.
@zimbatm
Copy link
Contributor Author

zimbatm commented Dec 29, 2022

I think it's fixed but there is an issue with nix profiles

@domenkozar domenkozar merged commit b15708f into cachix:master Dec 30, 2022
@zimbatm zimbatm deleted the github-access-token branch December 30, 2022 19:00
@zimbatm zimbatm restored the github-access-token branch December 30, 2022 19:00
@zimbatm zimbatm deleted the github-access-token branch December 30, 2022 19:00
@zimbatm zimbatm restored the github-access-token branch December 30, 2022 19:00
@NickHu
Copy link

NickHu commented Jan 4, 2023

Would love to see a new release for this action, so I can more readily use it, as I'm seeing the same issues! Thanks for your contribution!

@domenkozar
Copy link
Member

Couldn't we just set it by default?

@zimbatm
Copy link
Contributor Author

zimbatm commented Jan 4, 2023

AFAIK secrets.GITHUB_TOKEN has to be passed explicitly. But this could become the default example in the README and nix.dev.

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.

None yet

3 participants