Skip to content
This repository was archived by the owner on Mar 4, 2026. It is now read-only.

Cleanup: Remove validator#499

Closed
schmidt-sebastian wants to merge 1 commit intomrschmidt-cleanupfrom
mrschmidt-novalidator
Closed

Cleanup: Remove validator#499
schmidt-sebastian wants to merge 1 commit intomrschmidt-cleanupfrom
mrschmidt-novalidator

Conversation

@schmidt-sebastian
Copy link
Copy Markdown
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
Copy Markdown
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 subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

4 participants