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

fix(common): class-validator interface v0.10.1 #3059

Merged

Conversation

ruscon
Copy link

@ruscon ruscon commented Sep 28, 2019

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

@ruscon ruscon changed the title fix(common): update class-validator interface fix(common): class-validator interface v0.10.1 Sep 28, 2019
@coveralls
Copy link

Pull Request Test Coverage Report for Build 3076

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.349%

Totals Coverage Status
Change from base Build 2982: 0.0%
Covered Lines: 4141
Relevant Lines: 4343

💛 - Coveralls

@kamilmysliwiec kamilmysliwiec added this to the 6.8.0 milestone Sep 30, 2019
@kamilmysliwiec kamilmysliwiec changed the base branch from master to 6.8.0 September 30, 2019 08:13
@kamilmysliwiec kamilmysliwiec merged commit 31f85a7 into nestjs:6.8.0 Sep 30, 2019
@kamilmysliwiec
Copy link
Member

Thank you!

@kamilmysliwiec kamilmysliwiec mentioned this pull request Sep 30, 2019
3 tasks
@ruscon ruscon deleted the fix/class-validator-external-interface branch September 30, 2019 09:34
Copy link
Contributor

@prateekkathal prateekkathal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor grammatical errors... 😄

@@ -42,7 +52,7 @@ export interface ValidatorOptions {
value?: boolean;
};
/**
* Setting true will cause failed validation of unknown objects.
* Settings true will cause fail validation of unknown objects.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kamilmysliwiec Not sure why this was changed. The grammar doesn't sound correct 😅

@@ -25,7 +35,7 @@ export interface ValidatorOptions {
groups?: string[];
/**
* If set to true, the validation will not use default messages.
* Error message will always be undefined if it's not explicitly set.
* Error message always will be undefined if its not explicitly set.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Error messages will always be undefined if they are not explicitly set

@lock
Copy link

lock bot commented Feb 5, 2020

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 Feb 5, 2020
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.

4 participants