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

[#6630] Fix CI builds #6631

Merged
merged 1 commit into from Nov 2, 2021
Merged

[#6630] Fix CI builds #6631

merged 1 commit into from Nov 2, 2021

Conversation

gbp
Copy link
Member

@gbp gbp commented Nov 2, 2021

Relevant issue(s)

Fixes: #6630

What does this do?

Update theme repo URLs

Why was this needed?

Switch from the git protocol to https. This is required as GitHub
are strengthening their security around the use of the git protocol.

See: https://github.blog/2021-09-01-improving-git-protocol-security-github/

Implementation notes

Screenshots

Notes to reviewer

Switch from the `git` protocol to `https`. This is required as GitHub
are strengthening their security around the use of the git protocol.

Fixes: #6630
See: https://github.blog/2021-09-01-improving-git-protocol-security-github/
@gbp
Copy link
Member Author

gbp commented Nov 2, 2021

It seems Code Climate fetch is also failing due to this

@gbp
Copy link
Member Author

gbp commented Nov 2, 2021

Have tested and made this change to our servers repo.

@gbp gbp merged commit 1efe36a into develop Nov 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI builds failing to install theme
1 participant