aristanetworks / cloudvision

Resources and documentation for Arista's CloudVision platform
21 stars 13 forks source link

fix(rollup): fix incorrect rollup configs #945

Open kaiyoma opened 1 year ago

kaiyoma commented 1 year ago

The Rollup config for cloudvision-connector references the replace plugin, but this is the wrong package to do that, since that package never uses process.env in its source code. This config belongs in a-msgpack, which does access process.env in its source.

This fix will remove process.env from appearing in the build output of a-msgpack, where it appears currently and causes non-webpack callers to crash because process is not defined in a browser.