search

geo-smart / geo-smart.github.io

Organization website and blog for GeoSMART, dedicated to expanding the role of ML in the geosciences.
https://geo-smart.github.io/
1 stars 2 forks source link

The build script `build.js` is broken #1

Closed StefanTodoran closed 1 year ago

StefanTodoran commented 1 year ago

Any change to site structure (new html files, removal of html files, or refactoring of html files) will immediately cause production.js to fail catastrophically.

The script also does not clear out the docs folder prior to running. This is an issue affecting both the mirroring of structure changes which involve removal of html files, but also for asset folder removals which are significantly more frequent. This is a silent issue in that nothing will break but it persists unnecessary bloat which could be deleted.

StefanTodoran commented 1 year ago

UPDATE: The script name has been changed to build.js for easier autocomplete. The script was also reworked, fixing the clearing out of files. Asset removals should now be automatic (testing pending).

StefanTodoran commented 1 year ago

The build script now grabs all .js, .html and .css files, so this issue is fixed.

The one remaining improvement might be to not require the .js files to be in the js folder and likewise for the .css files, however this is good site structure and shouldn't be changed so it can stay.