search

IBM / report-toolkit

A toolkit for consumers of Node.js diagnostic Reports
https://ibm.github.io/report-toolkit
Other
84 stars 10 forks source link

make transformer initialization less horrid #41

Closed boneskull closed 5 years ago

boneskull commented 5 years ago

"The configuration stuff could be some of the worst code I've ever written. There's more refactoring to be done here; we should load the transformers first using defaults & user-supplied configuration, THEN we should validate the chain. Right now we're trying to do both at once which is just vile"

boneskull commented 5 years ago

so, load first, then I think we can defer chain validation to the "run" bit