Closed baywet closed 5 months ago
@gavinbarron @musale I'd love your input on this one when you have a couple of minutes.
I agree with removing cjs support all together. You have made valid points to remove it. ESM has more desirable benefits like tree-shaking which helps developers bundle smaller sizes of the sdk in their apps. In the core https://github.com/microsoftgraph/msgraph-sdk-typescript-core repo, this has already been done. In mgt, we removed it and directed using module syntax in the browser when loading mgt from the cdn https://github.com/microsoftgraph/microsoft-graph-toolkit/releases/tag/v4.0.0.
@sebastienlevert I think we have an agreement on the engineering side for this one. Especially since it's blocking other work. Any opposition to removing common is support here?
No. We should definitely drop it at this point.
Thanks for your input. @musale please proceed as part of your existing PR.
Decision is to drop commonjs support.
Wait, we still need to do some work to remove it
Since we originally stood up this library the ecosystem has moved a lot:
IMHO we should remove commonjs support after testing our main scenarios (SPFx, TeamsFX....) and BEFORE the GA so people don't end up in confusing variants + configuration misalignment issues.