fix: resolve most comments

Signed-off-by: David van der Spek <vanderspek.david@gmail.com>
This commit is contained in:
David van der Spek 2023-06-27 12:27:05 +02:00
parent 0f846853fe
commit 6a5846b32e
Failed to extract signature

View file

@ -14,20 +14,21 @@ import (
) )
var ( var (
// ErrStopReposWalk is used as a return value to indicate that the repository path walk // ErrStopReposWalk is a sentinel error to indicate that the repository path walk was stopped.
// should be stopped. It's not returned as an error by any function.
ErrStopReposWalk = errors.New("stop repos walk") ErrStopReposWalk = errors.New("stop repos walk")
) )
// Returns a list or a partial list of repositories in the registry. // Repositories populates the passed passed repos slice with repositories in the
// registry up to the capacity of the slice. Returns the number of repos returned and
// `io.EOF` if no more repositories are available.
// Because it's a quite expensive operation, it should only be used when building up // Because it's a quite expensive operation, it should only be used when building up
// an initial set of repositories. // an initial set of repositories.
func (reg *registry) Repositories(ctx context.Context, repos []string, last string) (n int, err error) { func (reg *registry) Repositories(ctx context.Context, repos []string, last string) (n int, err error) {
var finishedWalk bool var finishedWalk bool
var foundRepos []string var foundRepos []string
if len(repos) == 0 { if len(repos) == 0 && cap(repos) == 0 {
return -1, errors.New("no repos requested") return 0, errors.New("no repos requested")
} }
root, err := pathFor(repositoriesRootPathSpec{}) root, err := pathFor(repositoriesRootPathSpec{})