rladies / rladies.github.io

R-Ladies Hugo Website
Other
11 stars 20 forks source link

hugo server settings by type #203

Closed drmowinckels closed 1 year ago

drmowinckels commented 1 year ago

Moving some build settings into the configuration of hugo instead of hardcoded in github workflow. This way, when people work locally, hugo will behave the same for all, not needing to rely on specific build commands.

github-actions[bot] commented 1 year ago

:tada: The preview is built! Check it out

cosimameyer commented 1 year ago

Not sure if it’s on purpose but the build & deploy (production) job was skipped (since we’ve changes in the production .tomls (as far as I’ve seen), I’m wondering if we shouldn’t check it as well). If not, I’m absolutely fine with merging it 🙌

drmowinckels commented 1 year ago

production should be skipped in a PR, but the preview is built in stead. then when merged, hopefully, production will be run but preview skipped :)

drmowinckels commented 1 year ago

But I see what you mean! Let me have a think on how we can do that. for now this should be good. but we should have a think if we have a better way of testing production (as we have seen it fail)