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 yarn installation issues #50804

Closed
wants to merge 1 commit into from
Closed

Conversation

fheinecke
Copy link
Contributor

Configures yarn to use https instead of ssh for git clone operations.

GH appears to have released either a regression, or yet another breaking change with no announcement. This works around the issue while GH sorts their problems out.

Signed-off-by: Fred Heinecke <[email protected]>
@fheinecke fheinecke added backport-required no-changelog Indicates that a PR does not require a changelog entry labels Jan 6, 2025
@github-actions github-actions bot requested review from greedy52 and ryanclark January 6, 2025 22:24
@fheinecke fheinecke enabled auto-merge January 6, 2025 23:06
@benarent
Copy link
Contributor

benarent commented Jan 7, 2025

The lint docs issues should be fixed via the correct plugin, but it looks like it's still pulling an older version.

@@ -23,6 +23,8 @@ set -e
cp /etc/teleport/certs/rootCA.pem /usr/local/share/ca-certificates/teleport.crt
update-ca-certificates

git config --global url."https://github".insteadOf ssh://git@github
git config --global url."https://github.com/".insteadOf [email protected]:s
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is the trailing "s" intentional?

@fheinecke
Copy link
Contributor Author

This shouldn't be needed anymore. Fixed in gravitational/docs-website#100

@fheinecke fheinecke closed this Jan 7, 2025
auto-merge was automatically disabled January 7, 2025 19:02

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-required no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants