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

Add support for custom validators #96

Merged
merged 2 commits into from
Sep 4, 2014
Merged

Add support for custom validators #96

merged 2 commits into from
Sep 4, 2014

Conversation

sterling
Copy link
Contributor

@sterling sterling commented Sep 4, 2014

No description provided.

@ctavan
Copy link
Member

ctavan commented Sep 4, 2014

Thanks for the patch, looks cool.

However could you please separate the whitespace cleanup and the functional change into two separate commits?

@sterling
Copy link
Contributor Author

sterling commented Sep 4, 2014

I've broken up the commit into two.

ctavan added a commit that referenced this pull request Sep 4, 2014
Add support for custom validators
@ctavan ctavan merged commit 83da166 into express-validator:master Sep 4, 2014
@ctavan
Copy link
Member

ctavan commented Sep 4, 2014

Published to npm as

+ express-validator@2.4.0

@sterling
Copy link
Contributor Author

sterling commented Sep 4, 2014

Great, thanks!

@lock
Copy link

lock bot commented May 31, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators May 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants