search

citp / news-disinformation-study

A research project on how web users consume, are exposed to, and share news online.
8 stars 2 forks source link

correct path for output file, to prevent stripping characters from th… #98

Closed rhelmer closed 3 years ago

rhelmer commented 3 years ago

…e beginning of scripts

We found during testing that worker scripts where getting characters removed from the beginning of the filename, for instance: src/aggregateStatistics.worker.js -> dist/gregateStatistics.worker.js

I'm pretty sure this is because the roll-up config is expecting to strip "study" from the origin filename, and not "src".

This is working locally for me, and to unblock release activities I landed this on a release branch and tagged it for 2.0.1, I'll make sure any changes on that branch are cherry-picked here as well.

jonathanmayer commented 3 years ago

Good catch, I introduced this bug yesterday when updating the directory structure to match the Rally study template (study/ → src/).