Skip to content
This repository was archived by the owner on Sep 11, 2020. It is now read-only.

transport: ssh, NewPublicKeys helper #347

Merged
merged 1 commit into from
Apr 21, 2017
Merged

transport: ssh, NewPublicKeys helper #347

merged 1 commit into from
Apr 21, 2017

Conversation

mcuadros
Copy link
Contributor

@mcuadros mcuadros commented Apr 18, 2017

Fixes #342

@mcuadros mcuadros requested review from alcortesm and smola April 18, 2017 07:41
@@ -13,6 +14,8 @@ import (
"golang.org/x/crypto/ssh/agent"
)

const DefaultSSHUsername = "git"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe just DefaultUsername? It's already ssh package.

@codecov
Copy link

codecov bot commented Apr 21, 2017

Codecov Report

Merging #347 into master will decrease coverage by 0.06%.
The diff coverage is 57.89%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #347      +/-   ##
==========================================
- Coverage   76.69%   76.63%   -0.07%     
==========================================
  Files         122      122              
  Lines        8441     8449       +8     
==========================================
+ Hits         6474     6475       +1     
- Misses       1258     1262       +4     
- Partials      709      712       +3
Impacted Files Coverage Δ
plumbing/transport/ssh/auth_method.go 30.61% <57.89%> (+3.33%) ⬆️
references.go 73.22% <0%> (-2.75%) ⬇️

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 a41491f...373e597. Read the comment docs.

@mcuadros mcuadros merged commit d324899 into src-d:master Apr 21, 2017
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants