Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Fixes failing CI #25629

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fixes failing CI #25629

wants to merge 1 commit into from

Conversation

darangi
Copy link
Contributor

@darangi darangi commented Sep 22, 2022

  • The CI currently fails due to the following
    • An outdated windows image
    • Rate limiting on the GitHub API

@DeeDeeG
Copy link
Contributor

DeeDeeG commented Sep 28, 2022

(Note: It seems like the official repo has got the GitHub API rate limiting fixed in their CI, but I comment in case anyone is trying to run CI on a fork and getting stuck on this.)

If I understand correctly, the GitHub API rate limiting should be fixed if you set a valid GitHub API token as the GITHUB_TOKEN environment variable, in the CI environment.

Per this note in the vscode-ripgrep docs: https://github.com/microsoft/vscode-ripgrep/blob/v1.14.2/README.md#github-api-limit-note

(How to create a GitHub API token: https://docs.github.com/en/authentication/keeping-your-account-and-data-secure/creating-a-personal-access-token -- no permissions needed, just public API access, so don't need to check any permissions boxes, at least for the Pull Requests pipeline.)

(How to set the token in Azure Pipelines UI: https://learn.microsoft.com/en-us/azure/devops/pipelines/process/set-secret-variables?view=azure-devops&tabs=yaml%2Cbash#secret-variable-in-the-ui)

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

Successfully merging this pull request may close these issues.

None yet

2 participants