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

Simplify namespace structure #14

Closed
cmeeren opened this issue Apr 6, 2019 · 2 comments
Closed

Simplify namespace structure #14

cmeeren opened this issue Apr 6, 2019 · 2 comments

Comments

@cmeeren
Copy link
Contributor

cmeeren commented Apr 6, 2019

While testing in #12 (comment), I noticed that I had to open separate namespaces to get the result and asyncResult instances, and yet another one (the main FsToolkit.ErrorHandling) to get the helpers (Result.xxx etc.).

I suggest that we make result and asyncResult available in the main namespace (perhaps other CEs too?). While I may be biased to my own usage, I guess that people using this library are very likely to want these builders, so we shouldn't require them to have three opens in every relevant file.

(I haven't investigated the rest of the namespace/module structure of FsToolkit.ErrorHandling; there might be other places we could simplify, too.)

@demystifyfp
Copy link
Owner

I agree. Will incorporate it in the new release.

demystifyfp pushed a commit that referenced this issue Apr 10, 2019
@demystifyfp
Copy link
Owner

Fixed in 1.0.0

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

No branches or pull requests

2 participants