MaybeJustJames / zephyr

Tree shaking breeze for PureScript CoreFn AST
Mozilla Public License 2.0
159 stars 15 forks source link

Run DCE on ReExports from a module as well as exports #70

Closed MaybeJustJames closed 2 years ago

MaybeJustJames commented 2 years ago

Purs 0.14 series added a moduleReExports field to the Module type which was initially left intact. This caused a problem when re-exported names were eliminated in the code but left in the re-export list. This commit eliminates symbols from the re-export list too and adds a test.

MaybeJustJames commented 2 years ago

Superseded by #71