stackbuilders / dotenv-hs

Load environment variables from dotenv files for Haskell
https://hackage.haskell.org/package/dotenv
MIT License
65 stars 14 forks source link

Updating CI to new `get-tested` version (v0.1.7.0) #191

Closed DavidMazarro closed 8 months ago

DavidMazarro commented 8 months ago

I opened this issue https://github.com/Kleidukos/get-tested/issues/32 in the get-tested repo with a feature request to support setting the specific version of the OS runner we want to run; prior to the v0.1.7.0 version, the runners were all set to *-latest. Thanks to @Kleidukos' work, this is now supported in the most recent v0.1.7.0 version of get-tested. This PR changes our workflow to this new version of the action.

DavidMazarro commented 8 months ago

I've set the macos-version to 14 for now with two purposes:

  1. Testing that the new version of the action works as expected
  2. Out of curiosity, testing the Apple Silicon runner (and see if the library builds successfully) After that, I'll revert it to macos-latest. It would be interesting to ensure compatibility with Apple Silicon, but the runner is still in beta, so I don't think it's advisable using it for now.
DavidMazarro commented 8 months ago

Probably not a big surprise, but the Apple Silicon macOS runner doesn't play nicely with GHC 8; it complains about the version of LLVM. When run just with GHC 9.6 and 9.8, it works fine and the library builds without issues.

After testing that, I have reverted back to the latest runner.

DavidMazarro commented 8 months ago

I think the CI is currently failing because some dependency cache for the Apple Silicon runner was created, and now the macos-latest runner is having issues building dependencies with that dependency cache. I'll take a look into it.

Edit: fixed (by invalidating the ill cache).

CristhianMotoche commented 1 month ago

@all-contributors please add @DavidMazarro for code

allcontributors[bot] commented 1 month ago

@CristhianMotoche

I've put up a pull request to add @DavidMazarro! :tada: