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

[ci] Use GitHub token to avoid ripgrep download issue #1037

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

marcdumais-work
Copy link
Contributor

@marcdumais-work marcdumais-work commented Dec 15, 2023

In a recent CI run, I noticed this error:
"Downloading ripgrep failed: Error: Request failed: 403"

This can randomly happen on CI machines, because GitHub has limits when anonymously accessing its API, which we do when downloading ripgrep. There's an easy fix: use an auto-generated GitHub token when building the Theia app.

Also tidied-up the workflow file a little, to make it easier to read.

@marcdumais-work marcdumais-work force-pushed the ci-use-github_token branch 3 times, most recently from 174593a to 75b1e31 Compare December 15, 2023 15:32
In a recent CI run, I noticed this error:
"Downloading ripgrep failed: Error: Request failed: 403"

This happens because of GitHub has limits when anonymously accessing
its API, which we do when downloading ripgrep. There's an easy fix:
use an auto-generated GitHub token when building the Theia app.

Also tidied-up the workflow file a little, to make it easier to read.

Signed-off-by: Marc Dumais <[email protected]>
@marcdumais-work
Copy link
Contributor Author

marcdumais-work commented Dec 15, 2023

While in there, I have cleaned-up the labels used for the workflow, i.e. its name and the jobs and step names. I think it will look nicer in the PR "checks" section and also in the detailed Actions page. Let me know if you do not like.

Copy link
Collaborator

@bhufmann bhufmann left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks!

@marcdumais-work marcdumais-work merged commit dfa77bd into master Dec 15, 2023
3 checks passed
@marcdumais-work marcdumais-work deleted the ci-use-github_token branch December 15, 2023 19:06
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.

2 participants