the only reason that ConsoleContext was still used was because the options hook doesn't support certain context functions like this.error / this.warn etc
but buildStart does! so we can just move pretty much everything into buildStart instead
didn't move setting rollupOptions because that value gets hashed and the value in buildStart is different, as it is after all plugins' options hooks have ran
I'm not sure which is the better one to hash, but I left it as is so that the hash remains the same
this way we don't need ConsoleContext what-so-ever and so can remove it entirely!
as well as IContext, its tests, etc
use this.error instead of throwing errors in parse-tsconfig as it exists now
couldn't in the options hook, can in buildStart
this also ensure we don't have all the rpt2: prefixes ever missing again
Summary
Remove
ConsoleContext
(and the need for it) by using thebuildStart
hook instead of theoptions
hookDetails
the only reason that
ConsoleContext
was still used was because theoptions
hook doesn't support certain context functions likethis.error
/this.warn
etcbuildStart
does! so we can just move pretty much everything intobuildStart
insteadrollupOptions
because that value gets hashed and the value inbuildStart
is different, as it is after all plugins'options
hooks have ranConsoleContext
what-so-ever and so can remove it entirely!IContext
, its tests, etcuse
this.error
instead ofthrow
ing errors inparse-tsconfig
as it exists nowoptions
hook, can inbuildStart
rpt2:
prefixes ever missing againparse-tsconfig.spec
to account for this change