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

Rename ClientM to Client #467

Closed
3noch opened this Issue Apr 13, 2016 · 7 comments

Comments

Projects
None yet
4 participants
@3noch

3noch commented Apr 13, 2016

In #434 we added an alias called Handler for ExceptT ServantErr IO. We already have a similar client-side alias called ClientM. It seemed people agreed we should change ClientM to Client to be consistent with Handler.

@soenkehahn

This comment has been minimized.

Contributor

soenkehahn commented Apr 15, 2016

IIRC there was a consensus that ClientM should be renamed to something better.

Client doesn't really work, because we already have the type family Client.

@jkarni

This comment has been minimized.

Member

jkarni commented May 10, 2016

Are there any concrete proposals, given that Client doesn't work, or should we close this?

@3noch

This comment has been minimized.

3noch commented May 10, 2016

  1. Requester
  2. ClientApp
  3. WebClient
@soenkehahn

This comment has been minimized.

Contributor

soenkehahn commented May 11, 2016

I'd like to throw in ClientAction and ClientCommand. Generally I'm very un-opinionated about this. I'd be fine with closing this too.

@jkarni

This comment has been minimized.

Member

jkarni commented May 13, 2016

I myself prefer ClientM, but also don't have strong opinions.

@jkarni

This comment has been minimized.

Member

jkarni commented Sep 8, 2016

ClientM is actually a monad now - should we close?

@fizruk

This comment has been minimized.

Member

fizruk commented Jan 20, 2017

Closing this, feel free to reopen if I missed something.

@fizruk fizruk closed this Jan 20, 2017

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