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

Display All Validation Errors #152

Conversation

Animesh-Ghosh
Copy link
Contributor

Display width and height validations errors together if both are invalid.

This is related to #146.

Display width and height validations errors together if both are invalid.
@igorkasyanchuk
Copy link
Owner

@Animesh-Ghosh can you add a simple test for this PR?

@Animesh-Ghosh
Copy link
Contributor Author

We can also add simplecov to check the code coverage.
I had added it as a development dependency while writing the test and it seems the code coverage on master is 88.85% and after this test was added it increase to 89.32%.

Let me know if this is something that might be needed.

@igorkasyanchuk
Copy link
Owner

@Animesh-Ghosh simplecov would be nice to add. Please add it as a separate PR

@igorkasyanchuk igorkasyanchuk merged commit a1a706a into igorkasyanchuk:master Apr 17, 2022
@Animesh-Ghosh Animesh-Ghosh deleted the display-all-validation-errors branch December 13, 2023 07:25
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