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

improve valid result, you can set success or error message when valid… #231

Merged
merged 3 commits into from
Aug 24, 2016

Conversation

kuaisheng
Copy link

you can return message(success or error) in expression Functions,
like {
result: false,
message: 'input should not include xxxx'
}

@coveralls
Copy link

coveralls commented Aug 23, 2016

Coverage Status

Coverage remained the same at 93.73% when pulling 13068f1 on kuaisheng:master into c5e2603 on huei90:master.

@hueitan
Copy link
Owner

hueitan commented Aug 23, 2016

Thanks and Please also update the following

  • Test file
  • Grunt build
  • API Documentation

@coveralls
Copy link

coveralls commented Aug 24, 2016

Coverage Status

Coverage decreased (-0.1%) to 93.617% when pulling bcf3881 on kuaisheng:master into c5e2603 on huei90:master.

@kuaisheng
Copy link
Author

They are completed yet
Thanks

@coveralls
Copy link

coveralls commented Aug 24, 2016

Coverage Status

Coverage decreased (-0.1%) to 93.617% when pulling 73fc9c1 on kuaisheng:master into c5e2603 on huei90:master.

@hueitan
Copy link
Owner

hueitan commented Aug 24, 2016

looks good 👍 Thanks! This is additional feature.

@hueitan hueitan merged commit 531cb12 into hueitan:master Aug 24, 2016
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.

3 participants