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

Cleanup: Remove validator #499

Closed
wants to merge 1 commit into from

Conversation

schmidt-sebastian
Copy link
Contributor

@schmidt-sebastian schmidt-sebastian commented Dec 28, 2018

This is part of #512 but can be reviewed independently.

This removes the Validator class and replaces the remaining invocations with free-standing methods.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 28, 2018
@schmidt-sebastian schmidt-sebastian force-pushed the mrschmidt-novalidator branch 6 times, most recently from 1c33538 to ec5e463 Compare December 28, 2018 20:57
@JustinBeckwith JustinBeckwith added the 🚨 This issue needs some love. label Feb 7, 2019
@yoshi-automation yoshi-automation added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Feb 7, 2019
@JustinBeckwith JustinBeckwith added the needs work This is a pull request that needs a little love. label Feb 9, 2019
@yoshi-automation yoshi-automation removed the 🚨 This issue needs some love. label Feb 9, 2019
@schmidt-sebastian
Copy link
Contributor Author

Merged as part of #512

@schmidt-sebastian schmidt-sebastian deleted the mrschmidt-novalidator branch March 11, 2019 03:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement. needs work This is a pull request that needs a little love.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants