Skip to content

Conversation

@lafriks
Copy link
Member

@lafriks lafriks commented Jul 30, 2017

Fixes #2232

@lafriks lafriks added this to the 1.2.0 milestone Jul 30, 2017
@lafriks lafriks force-pushed the fix_ssh_sockets branch 2 times, most recently from 04a6df3 to 03fd058 Compare July 30, 2017 19:42
Copy link
Member

Choose a reason for hiding this comment

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

Typo newInternalReqest

Copy link
Member Author

Choose a reason for hiding this comment

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

@silverwind fixed typo

@lunny
Copy link
Member

lunny commented Aug 3, 2017

LGTM, but integration test is better.

@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Aug 3, 2017
@lafriks
Copy link
Member Author

lafriks commented Aug 3, 2017

Can not create integration test for this as it requires external ssh server

@sapk
Copy link
Member

sapk commented Aug 3, 2017

LGTM

@tboerger tboerger added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Aug 3, 2017
@lunny lunny merged commit fa8d40f into go-gitea:master Aug 3, 2017
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UpdatePublicKey: unsupported protocol scheme "unix" when pushing via ssh

5 participants