openlayers / ol-cesium

OpenLayers - Cesium integration
http://openlayers.org/ol-cesium/
BSD 2-Clause "Simplified" License
968 stars 321 forks source link

fix: parcel serve target dist directory #1197

Closed pedroslvieira closed 2 weeks ago

pedroslvieira commented 1 month ago

The examples were not working on dev environment after running npm start because there is a bug in parcel that is ignoring the distDir attribute in targets for parcel serve. I replaced with cli commands and added a prestart hook with the commands that were in build-examples to add parts that are not compiled by parcel (should we also investigate that?).

bug reports in parcel: https://github.com/parcel-bundler/parcel/issues/8715 https://github.com/parcel-bundler/parcel/issues/8105 https://github.com/parcel-bundler/parcel/discussions/6778

Also, should I just remove the targets object from the code?

gberaudo commented 2 weeks ago

Thanks again!