-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Some changes to the CI script #55
base: develop
Are you sure you want to change the base?
Conversation
Partly rendered obsolete by #56, but not entirely. |
1. Restore a `Data.Semigroup.Commutative` reexport accidentally removed in #56. (I accidentally forgot to notice during review.) 2. Replace partial `Map.lookup` with proper custom error for internal error. (This would make debugging a bug in the implementation easier.) This was part of #55. Co-Authoredy-By: Dmitry Olshansky <olshanskydr@gmail.com>
e48ce5f
to
c0ac897
Compare
Rebased, now just CI changes |
@@ -13,32 +13,52 @@ jobs: | |||
|
|||
name: GHC ${{ matrix.ghc }} on ${{ matrix.os }} | |||
steps: | |||
- uses: actions/checkout@v3 | |||
- uses: haskell/actions/setup@v2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
haskell/actions
OK yes that repo is deprecated, I just learned. This helps me understand why these changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My main goal was compiling with javascript-backend
. I think that #56 do it so I am OK to close...
No description provided.