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

future: imported from dcos-storage/pkg/future #69

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jdef
Copy link
Contributor

@jdef jdef commented Jul 26, 2019

future

Checklist

  • [?] ~80% unit test coverage?
  • Updated README.md?
  • Completed sections of this PR template?
  • Edited all sections [IN BRACKETS]

Overview of Change

future is a utility package that provides helpful functionality on top of a basic Go channel.
This package has been incubating in the dcos-storage project and we're upstreaming it now so that other projects can leverage it.

Changelog [optional]

N/A

Affected Usage

N/A

@jdef jdef requested a review from gpaul July 26, 2019 18:39
@jdef
Copy link
Contributor Author

jdef commented Jul 26, 2019

ouch, lots of missing docs. marking this WIP and adding docs now...

/go/src/github.com/dcos/dcos-go/future/future.go:11:6: exported type Interface should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:15:6: exported type Func should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:17:1: exported method Func.Future should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:26:6: exported type Future should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:108:6: exported type Completer should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:112:6: exported type CompleterFunc should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:114:1: exported method CompleterFunc.Complete should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:116:1: exported method CompleterFunc.Apply should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:118:1: exported method CompleterFunc.If should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:127:1: exported function Do should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:131:6: exported type Options should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:137:6: exported type Optional should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:141:6: exported type Option should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:143:1: exported method Option.Apply should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:145:6: exported type Promise should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:198:1: exported function NewPromise should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:354:1: exported function Nil should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:406:6: exported type Merger should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:413:1: exported method MergeFunc.Merge should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:423:1: exported method FirstError.Merge should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/future.go:588:5: exported var ErrFlattenInvalid should have comment or be unexported
/go/src/github.com/dcos/dcos-go/future/must.go:7:1: exported function Must should have comment or be unexported

@jdef
Copy link
Contributor Author

jdef commented Jul 26, 2019

unrelated dcos-go-windows error:

go: github.com/Sirupsen/[email protected]: parsing go.mod: unexpected module path "github.com/sirupsen/logrus"
go: error loading module requirements
Failed to install github.com/Sirupsen/logrus
at Install-GoDependencies, C:\Users\jenkins\workspace\public-dcos-go-windows\run-go-tests.ps1: line 219
at <ScriptBlock>, C:\Users\jenkins\workspace\public-dcos-go-windows\run-go-tests.ps1: line 462

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant