Remove require#11689
Conversation
|
@jna4k I notice this PR is open, still in draft, since mid November. If you don't have plans to update it and make it ready for review in the next few days I would like to close it so that you can reopen it when you have done. Thank you for your understanding. |
|
Hey, my intentions was to get back to this PR, but then the holidays and other stuff at work got in the way. I still intend to finish what i started here, but unfortunately it will have to wait a while longer. For now I am okay with closing this PR, and I will make a new one, once the code is ready. |
Thank you very much for your feedback and for your intention to finalize it. We are looking forward to receiving your updates as soon as possible. In the meantime, I would like to take this opportunity to ask you to visit the updated contributing rules in the project's WIKI where you can see info related to the CLA that is necessary for contributing updates to the MS core. Thank you so much again for your contribution in evolving MS and the MS community. |
DRAFT - THIS PR IS STILL A DRAFT
I am still working on the PR, but there is still quite a bit of work left and I still need to do some more testing, but due to the amount of changes I thought it would put the PR up, even if its not quite ready for a review yet.
Description
We should change the codebase to use import/export instead of require.
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (check one with "x", remove the others)
Issue
What is the current behavior?
#11688
What is the new behavior?
Breaking change
Does this PR introduce a breaking change? (check one with "x", remove the other)
Other useful information