kentcdodds / webpack-validator-DEPRECATED

Use this to save yourself some time when working on a webpack configuration.
MIT License
93 stars 8 forks source link

External validators #20

Closed jonathanglasmeyer closed 8 years ago

jonathanglasmeyer commented 8 years ago

An approach discussed in #17 for solving the architectural question of how to integrate support for custom / not-webpack-core validators looks as follows:

Any thoughts on this? Will probably have some time on the weekend to tackle this.

Cheers, Jonathan

nyrosmith commented 8 years ago

This sounds very good to me but I think this should be very verbose of what's going on so the user gets an idea of what happens

jonathanglasmeyer commented 8 years ago

Sure. :+1: I'm not sure about the testing aspect. How would validator plugin authors test their stuff? There is some setup overhead involved right? I'm still wondering if the core project should include validator plugins in its tests or if that should be the responsibility of the validator plugin authors.

kentcdodds commented 8 years ago

Yeah, I think we'll have to write this in a modular way so we can have units that can accept stubbed out dependencies. Then we can have a handful of integration tests to make sure it all works together. I'm sure we can do this though :-)

jonathanglasmeyer commented 8 years ago

Will revisit this when we have a monorepo.