Norbert de Langen
Norbert de Langen
I don't have an estimate of when that suggestion will happen @joshwooding, so If you think this change will make an improvement, we should add the improvement for now 👍
Converted into draft, this seems to be a very pivotal package, not easy to convert. I might drop this for now, since I need to work on other stuff.
I think we should not have more peerDependencies, they do not actually fix the issue. What we should do instead is have the reference to the babel loader come from...
@IanVS I expanded upon my comment, maybe you hadn't seen it / got no notification, so just doing a ping here, in case.
@shilman I'd like your review on this.. I commented out the test code, because I didn't know if I should remove it or modify it, please advice what you'd like...
Discussed with @shilman he'll take a look at this, when he has time. I'll continue with other deprecations removal work.
This is one of the few remaining ones @tmeasday @shilman
@shilman Do you think it's possible for us to resolve this together on Monday? and get this merged?
Merge when green!
@anthonator could you please rebase on the `next` branch please? We cannot proceed with this PR otherwise. Thank you 🙏 !