lunny/git-lfs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Make api.Client instance local not public

Browse Source
This commit is contained in:
Steve Streeting 2016-11-28 15:54:10 +00:00
parent 092c879dbe
commit 4f6fdb8dc2
1 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
locking/locks.go
View File

@ -13,7 +13,7 @@ import (
var ( var (
// API is a package-local instance of the API client for use within // API is a package-local instance of the API client for use within
// various command implementations. // various command implementations.
API = api.NewClient(nil) apiClient = api.NewClient(nil)
// errNoMatchingLocks is an error returned when no matching locks were // errNoMatchingLocks is an error returned when no matching locks were
// able to be resolved // able to be resolved
errNoMatchingLocks = errors.New("lfs: no matching locks found") errNoMatchingLocks = errors.New("lfs: no matching locks found")
@ -37,13 +37,13 @@ func Lock(path, remote string) (id string, e error) {
return "", err return "", err
} }
s, resp := API.Locks.Lock(&api.LockRequest{ s, resp := apiClient.Locks.Lock(&api.LockRequest{
Path: path, Path: path,
Committer: api.CurrentCommitter(), Committer: api.CurrentCommitter(),
LatestRemoteCommit: latest.Sha, LatestRemoteCommit: latest.Sha,
}) })
if _, err := API.Do(s); err != nil { if _, err := apiClient.Do(s); err != nil {
return "", fmt.Errorf("Error communicating with LFS API: %v", err) return "", fmt.Errorf("Error communicating with LFS API: %v", err)
} }
@ -79,9 +79,9 @@ func Unlock(path, remote string, force bool) error {
// Unlock attempts to unlock a lock with a given id on the remote // Unlock attempts to unlock a lock with a given id on the remote
// Force causes the file to be unlocked from other users as well // Force causes the file to be unlocked from other users as well
func UnlockById(id, remote string, force bool) error { func UnlockById(id, remote string, force bool) error {
s, resp := API.Locks.Unlock(id, force) s, resp := apiClient.Locks.Unlock(id, force)
if _, err := API.Do(s); err != nil { if _, err := apiClient.Do(s); err != nil {
return fmt.Errorf("Error communicating with LFS API: %v", err) return fmt.Errorf("Error communicating with LFS API: %v", err)
} }
@ -134,8 +134,8 @@ func SearchLocks(remote string, filter map[string]string, limit int) *LockChanne
query := &api.LockSearchRequest{Filters: apifilters} query := &api.LockSearchRequest{Filters: apifilters}
QueryLoop: QueryLoop:
for { for {
s, resp := API.Locks.Search(query) s, resp := apiClient.Locks.Search(query)
if _, err := API.Do(s); err != nil { if _, err := apiClient.Do(s); err != nil {
errChan <- fmt.Errorf("Error communicating with LFS API: %v", err) errChan <- fmt.Errorf("Error communicating with LFS API: %v", err)
break break
} }
@ -177,13 +177,13 @@ func SearchLocks(remote string, filter map[string]string, limit int) *LockChanne
// If the API call is successful, and only one lock matches the given filepath, // If the API call is successful, and only one lock matches the given filepath,
// then its ID will be returned, along with a value of "nil" for the error. // then its ID will be returned, along with a value of "nil" for the error.
func lockIdFromPath(path string) (string, error) { func lockIdFromPath(path string) (string, error) {
s, resp := API.Locks.Search(&api.LockSearchRequest{ s, resp := apiClient.Locks.Search(&api.LockSearchRequest{
Filters: []api.Filter{ Filters: []api.Filter{
{"path", path}, {"path", path},
}, },
}) })
if _, err := API.Do(s); err != nil { if _, err := apiClient.Do(s); err != nil {
return "", err return "", err
} }