dillonkearns / elm-pages

Hybrid Elm framework with full-stack and static routes.
https://elm-pages.com
BSD 3-Clause "New" or "Revised" License
658 stars 97 forks source link

Convert `runInternalJob` to using a `switch` statement #467

Closed miniBill closed 7 months ago

netlify[bot] commented 7 months ago

Deploy request for elm-pages-todos pending review.

Visit the deploys page to approve it

Name Link
Latest commit e31b8995c157335cc7777640a7b328518ef57b07
dillonkearns commented 7 months ago

It was easier to make the switch on master than resolve the merge conflicts, especially with the code formatting changes. Thanks for the PR! The change is pushed with https://github.com/dillonkearns/elm-pages/commit/593312d159cb86b60b588aedcd9fe0edde347dfa 👍

miniBill commented 7 months ago

Yeah, it made little sense to spend a lot of time merging, good call!