exercism / purescript

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

Upgrade to PureScript 0.14.3 and add / update docs, remove references to bower #212

Closed icyrockcom closed 3 years ago

iHiD commented 3 years ago

(cc @emiel)

icyrockcom commented 3 years ago

@emiel Thanks!

  1. I'll close this PR and submit 3 separate PRs, so we discuss them one by one
  2. I'll look at purescript-test-runner and discuss in its repository as needed
  3. I'll restore bin/fetch-configlet
  4. Let me know how you'd like me to proceed regarding this if not the way I did it
ErikSchierboom commented 3 years ago

I'm not too familiar with configlet. It is generally run by CI but can be run locally as well. The bin/fetch-configlet script fetches the latest version. I'm not sure what is expected by Exercism developers here. It doesn't hurt to keep it I guess. Perhaps @ErikSchierboom can chime in her

configlet is indeed also intended to be run locally, as it not only has linting functionality but also syncing functionality that one runs locally to update the repo.

ErikSchierboom commented 3 years ago

Before this can be merged the purescript-test-runner needs to be updated as well.

Indeed, that is very important. Let me know if you need any help.