search

dimitri-xyz / haskell-coinbase-pro

A Haskell client for the Coinbase Pro API
MIT License
11 stars 6 forks source link

Update dependencies and verify build process #12

Open ericpashman opened 4 years ago

ericpashman commented 4 years ago

We're still on Stack LTS 9.5, which is now more than two years old. I have in mind a few specific changes that I want to make using functionality provided by newer versions of our dependencies.

I think there are also some issues getting the library to build with cabal-install alone. Let's make sure that works too after updating. I'll also try to remember to add a Nix config at some point.

ericpashman commented 4 years ago

The dependency set will be updated when #19 is merged. Let's check the cabal-install build process and close this.

dimitri-xyz commented 4 years ago

Apparently, I forgot to remove the dependency on the resource library from the .cabal file... :-(

ericpashman commented 4 years ago

Whoops, and I meant to run HLint to check for unused dependencies and imports aside from the ones I knew needed to go.

We should do a proper cleanup at some point. Maybe after getting all the tests passing would be a good time for that.

On May 19, 2020, at 01:05, Dimitri DeFigueiredo notifications@github.com wrote:

 Apparently, I forgot to remove the dependency on the resource library from the .cabal file... :-(

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.