-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Refactors config validation, deprecation and unification logic. #1805
Conversation
Fixes undefined prints in watch tests, also moves stderr to src.
0093463
to
afd30df
Compare
@lukastaegert I think I've solved it this time. Hahaha! |
This work looks great. We've been working on a TypeScript conversion in #1806, which has seen some inconsistencies in supporting the highly dynamic options interfaces. I wonder if this would simplify that or make it harder... |
@guybedford To solve the inconsistencies among the CLI and node I've created base input/output options in the file mergeOptions.js which is now the central place for all options related logic. All entry points now fallback to it. I hope this will make it simpler! |
This was originally raised for #1762. This builds on top of #1769