Skip to content
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

Refactor the code in order to be compatible with newer GHC #20

Merged
merged 6 commits into from
May 8, 2023

Conversation

hw202207
Copy link
Contributor

@hw202207 hw202207 commented Apr 28, 2023

  1. Fallback return to pure but not other way around
  2. Explicit TypeOperators
  3. Import Control.Monad to include MonadPlus explicitly

1. fallback return to pure but not other way around
2. explicit TypeOperators
3. import Control.Monad to include MonadPlus
@hw202207 hw202207 marked this pull request as ready for review April 28, 2023 22:12
@TOTBWF
Copy link
Owner

TOTBWF commented Apr 29, 2023

Thanks for putting in this works! Would you mind if I added new GHC versions to the build matrix as part of this PR?

@hw202207
Copy link
Contributor Author

I added ghc-9.6.1.
Happy to add others like 9.2 and 9.4 if you think is useful (in separated PR)

@hw202207
Copy link
Contributor Author

hw202207 commented May 8, 2023

Hi @TOTBWF, I updated the PR. Would you mind take another look and lmk any further problems.?

@TOTBWF
Copy link
Owner

TOTBWF commented May 8, 2023

Looks like we need to muck with CI a bit more; I can handle this, as it looks like it could be a bit involved.

@hw202207
Copy link
Contributor Author

hw202207 commented May 8, 2023

Thanks!

FYI:
I'm using github action plugin to setup GHC which seems simpler to me.

@TOTBWF TOTBWF merged commit e9dbb5f into TOTBWF:master May 8, 2023
4 checks passed
@TOTBWF
Copy link
Owner

TOTBWF commented May 8, 2023

Looks like CI is green! I'll make a hackage release later today, thanks so much!

@hw202207 hw202207 mentioned this pull request May 8, 2023
@Vekhir Vekhir mentioned this pull request Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants