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

Create extend() API so any Express module instance can be extended #5

Merged
merged 6 commits into from
Jul 29, 2013

Conversation

ericf
Copy link
Collaborator

@ericf ericf commented Jul 19, 2013

Previously, only the the Express module instance which is a peerDependency of this package would be extended with this package's functionality. This now exports the extend() function which allows any Express module instance to be extended.

Todo

  • Add tests
  • Get feedback on feature
  • Evaluate need for an app-level augment
  • Clean-up implementation
  • Evaluate whether augment() is safe for resProto, it might be shared between apps
  • Get sign-off from real world app usage

Previously, only the the Express module instance which is a
`peerDependency` of this package would be extended with this package's
functionality. This now exports the `extend()` function which allows
any Express module instance to be extended.
@ericf
Copy link
Collaborator Author

ericf commented Jul 19, 2013

@caridy and @redonkulus is this what you were looking for to be able to extend any Express module instance, not just the Express module instance which is a peerDependency?

@caridy
Copy link
Contributor

caridy commented Jul 20, 2013

Yeah, that will work. Can we get augment as well, just in case we get an express app instance without access to express directly?

@caridy
Copy link
Contributor

caridy commented Jul 24, 2013

LGTM

ericf added a commit that referenced this pull request Jul 29, 2013
Create `extend()` API so any Express module instance can be extended
@ericf ericf merged commit 9b9a81c into master Jul 29, 2013
@ericf ericf deleted the extend branch July 29, 2013 19:45
@caridy
Copy link
Contributor

caridy commented Jul 29, 2013

published in [email protected]

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