d3-selection circular dependency errors since switch to rollup

[alykat]

Seeing a lot of this in console since we switched over to rollup for bundling:

Circular dependency: ../graphics-js/node_modules/d3-selection/src/selection/index.js -> ../graphics-js/node_modules/d3-selection/src/selection/data.js -> ../graphics-js/node_modules/d3-selection/src/selection/index.js
Circular dependency: ../graphics-js/node_modules/d3-selection/src/selection/index.js -> ../graphics-js/node_modules/d3-selection/src/selection/data.js -> ../graphics-js/node_modules/d3-selection/src/selection/enter.js -> ../graphics-js/node_modules/d3-selection/src/selection/index.js
Circular dependency: ../graphics-js/node_modules/d3-selection/src/selection/index.js -> ../graphics-js/node_modules/d3-selection/src/selection/exit.js -> ../graphics-js/node_modules/d3-selection/src/selection/index.js
Circular dependency: ../graphics-js/node_modules/d3-selection/src/selection/index.js -> ../graphics-js/node_modules/d3-selection/src/selection/merge.js -> ../graphics-js/node_modules/d3-selection/src/selection/index.js
Circular dependency: ../graphics-js/node_modules/d3-selection/src/selection/index.js -> ../graphics-js/node_modules/d3-selection/src/selection/sort.js -> ../graphics-js/node_modules/d3-selection/src/selection/index.js

Seems to be annoying but benign? Will try the approach mentioned here to suppress the warning: d3/d3-selection#168

[thomaswilburn]

That's effectively what we ended up doing: Chalkbeat@1beab0f

Circular dependencies are allowed by ES modules, so it's not technically an error, and D3 is unlikely to change anytime soon.

[thomaswilburn]

(You may also want to pull the sync changes that I added after realizing that missing directories on either side would throw warnings: Chalkbeat@a7bdb68)

[alykat]

@thomaswilburn ah great! thank you! starting on that here — https://github.com/nprapps/dailygraphics-next/tree/ah-sync-crashes. will test and probably merge in the morning.