Skip to content

Rename expressify to express - #719

Merged
daffl merged 1 commit into
feathersjs:masterfrom
bertho-zero:patch-1
Nov 3, 2017
Merged

daffl merged 1 commit into
feathersjs:masterfrom
bertho-zero:patch-1

Conversation

@bertho-zero

Copy link
Copy Markdown
Contributor

According with https://github.com/feathersjs/express, and because I find it more appropriate, what do you think?

According with https://github.com/feathersjs/express, and because I find it more appropriate, what do you think?
@daffl

daffl commented Nov 3, 2017

Copy link
Copy Markdown
Member

That's how I had it before and someone else pointed out that it was confusing (because you'd expect that to be require('express')).

I also think we should keep it at express (especially since it has all its middleware on it).
Thanks!

@daffl
daffl merged commit 4840e0c into feathersjs:master Nov 3, 2017
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