search

exercism / purescript

Exercism exercises in PureScript.
https://exercism.org/tracks/purescript
MIT License
37 stars 32 forks source link

Port to spago (CI) #181

Closed emiel closed 3 years ago

emiel commented 3 years ago

Here is my attempt to continue with the work done here https://github.com/exercism/purescript/pull/127 in replacing bower/pulp with spago. I focused on getting CI setup to build and test the exercises (just practice exercises for now). The approach is more or less similar to what was being done for bower. Spago uses two files packages.dhall (defines package-set) and spago.dhall (project dependencies).

Please have a look (the last commit contains the interesting bits) and see if you think this approach is worth pursuing. If so I can continue to update the docs and add the finishing touches.

NOTE: Without sharing the output and .spago directories the CI run takes around 7 minutes...

@connorads @aimorris

emiel commented 3 years ago

Here is an example of a CI run: https://github.com/emiel/purescript/runs/2442322095?check_suite_focus=true

iHiD commented 3 years ago

@emiel Thank you 💙 @ErikSchierboom will take a look at this later in the week when he's back working. Before then hopefully some @exercism/purescript people can take a look! :)

SleeplessByte commented 3 years ago

The travis is failing because it's still using the old configlet

emiel commented 3 years ago

The travis is failing because it's still using the old configlet

Thanks for having a look @SleeplessByte. It is my understanding Excercism is migrating completely to GitHub CI and thus that is where my focus is now. I'd rather tackle configlet in another PR because I guess there is more work to do there. ;)

SleeplessByte commented 3 years ago

Totally. It was mostly informational and why I approved.

declarationperfume commented 3 years ago

I concur

emiel commented 3 years ago

Closing in favor of https://github.com/exercism/purescript/pull/185