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

Don't use git:// URLs in pre-commit hooks #699

Merged
merged 1 commit into from Mar 28, 2022
Merged

Conversation

mjpieters
Copy link
Contributor

Unauthenticated git:// URLs are no longer supported by GitHub:

    fatal: remote error: 
      The unauthenticated git protocol on port 9418 is no longer supported.
    Please see https://github.blog/2021-09-01-improving-git-protocol-security-github/ for more information.

@codecov
Copy link

codecov bot commented Mar 28, 2022

Codecov Report

Merging #699 (0f75124) into master (a90599e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #699   +/-   ##
=======================================
  Coverage   99.73%   99.73%           
=======================================
  Files           4        4           
  Lines         750      750           
  Branches      170      170           
=======================================
  Hits          748      748           
  Misses          2        2           
Flag Coverage Δ
unit 99.60% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a90599e...0f75124. Read the comment docs.

Unauthenticated `git://` URLs are no longer supported by GitHub:

```
    fatal: remote error:
      The unauthenticated git protocol on port 9418 is no longer supported.
    Please see https://github.blog/2021-09-01-improving-git-protocol-security-github/ for more information.
```
@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Mar 28, 2022
@mjpieters mjpieters merged commit 42a5e2a into master Mar 28, 2022
@mjpieters mjpieters deleted the unbreak_linting branch March 28, 2022 17:17
q0w pushed a commit to q0w/yarl that referenced this pull request Mar 29, 2022
Unauthenticated `git://` URLs are no longer supported by GitHub:

```
    fatal: remote error:
      The unauthenticated git protocol on port 9418 is no longer supported.
    Please see https://github.blog/2021-09-01-improving-git-protocol-security-github/ for more information.
```
webknjaz pushed a commit that referenced this pull request Aug 1, 2022
Unauthenticated `git://` URLs are no longer supported by GitHub:

```
    fatal: remote error:
      The unauthenticated git protocol on port 9418 is no longer supported.
    Please see https://github.blog/2021-09-01-improving-git-protocol-security-github/ for more information.
```

(cherry picked from commit 42a5e2a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot:chronographer:provided There is a change note present in this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant