-
Notifications
You must be signed in to change notification settings - Fork 34
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
replace mtl by transformers #62
Comments
Can you fix llvm-base as well? It doesn't make much sense to remove it from one location but not the other. |
LLVM.ST is now ported to transformers, as well. |
@amigalemming can you please give us a pull request so we can review the code changes etc? thanks |
The commit is here: I found the pull-requests via github very cumbersome (mix of shell and web access, looping javascripts at github, impossibility to remove commits), thus I am using code.haskell.org. |
do you want us to look at the patch or not? If yes, please take the 5 minutes to do a pull request. look forward to seeing the pull request. |
Is there a way to fork, making pull requests etc. via command line instead |
a cursory google search indicates http://stackoverflow.com/questions/4610530/github-cli-clients which in turn links to a bunch. |
After our last discussion in the mailing list I have replaced mtl by transformers in llvm-tf.
I propose to do the same replacement in 'llvm', too.
I have prepared a commit for this purpose, that you may simply pull in:
http://code.haskell.org/~thielema/llvm-pull/transformers/
'transformers' does not need any language extensions and does not export identifiers from Control.Monad and other 'base' modules.
The text was updated successfully, but these errors were encountered: