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

Namespace #34

Closed
kelunik opened this issue May 14, 2016 · 4 comments
Closed

Namespace #34

kelunik opened this issue May 14, 2016 · 4 comments
Milestone

Comments

@kelunik
Copy link
Member

kelunik commented May 14, 2016

Should we go with Interop\Async as namespace instead of Interop\Async\EventLoop?

I created a promise draft, using Interop\Async\Promise\Promise seems bad and as we just have a few interfaces, I think it'd be fine to use just Interop\Async as namespace for both.

@bwoebi
Copy link
Member

bwoebi commented May 14, 2016

I agree on that one, makes it easier to just import one namespace for all promisy and loopy stuff. (like promise transformation functions we'll probably provide (?))

also… less to type :-D

Interop\Async should provide its fundamental primitives at its main namespace - not need to split that up. (If we'd have naming conflicts, that'd be anyway bad as it makes importing harder/requires aliases - so that should be avoided anyway.)

@WyriHaximus
Copy link
Member

also… less to type :-D

IDE autocompletion 😛

@sagebind
Copy link

I'm in favor of a single, shorter namespace.

@kelunik
Copy link
Member Author

kelunik commented May 15, 2016

Looks like we have consensus here. 👍

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

No branches or pull requests

4 participants