Visitar URL original
Preparing for release: api and documentation by aspiwack · Pull Request #83 · tweag/capability · GitHub
Skip to content

Preparing for release: api and documentation - #83

Merged
aherrmann merged 10 commits into
masterfrom
api-and-documentation
Mar 9, 2020
Merged

aherrmann merged 10 commits into
masterfrom
api-and-documentation

Conversation

@aspiwack

@aspiwack aspiwack commented Mar 5, 2020

Copy link
Copy Markdown
Collaborator

I've tried to make the Haddock look good, basically.

@aspiwack
aspiwack requested review from aherrmann and gdeest March 5, 2020 15:07
Comment thread src/Capability/Derive.hs Resolved
Comment thread src/Capability/Derive.hs Resolved
Comment thread src/Capability/Derive.hs Resolved
aspiwack and others added 10 commits March 9, 2020 09:13
Since we are advertising to import capability modules unqualified,
having a type family like `All` exported in these module would pollute
the global namespace. This is a bad idea.

I don't think any of the function in `Capability.Constraint` are
needed in the normal operations of Capability. But if we decide that
one is, we should re-export that one specifically.
@aherrmann
aherrmann force-pushed the api-and-documentation branch from 198e8ad to 7ec5c4b Compare March 9, 2020 08:14
@aherrmann

Copy link
Copy Markdown
Contributor

I've rebased on master and force-pushed. Hopefully that fixes CI.

@aherrmann
aherrmann merged commit 481c086 into master Mar 9, 2020
@aherrmann
aherrmann deleted the api-and-documentation branch March 9, 2020 08:27
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.

2 participants