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

Fix package import error #153

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

JamieMaple
Copy link

应该是 Control.Monad.StateControl.Monad.Reader 不在 System

应该是 `Control.Monad.State` 和 `Control.Monad.Reader` 不在 `System` 下
@JamieMaple JamieMaple changed the title fix package import error Fix package import error Sep 14, 2018
Copy link

@SnowOnion SnowOnion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A correct correction
-
等一下,这个 PR 有 3 个 commit,我指的是 Fix typo in ch 19 和 Update 19.rst 那两个。

import 那个,则涉及到问题:是忠于原版翻译,还是对书中所有过时的代码进行翻新,还是翻新一部分?……

@JamieMaple 感谢贡献。看起来您是初学,RWH 这书确实比较老,可以先看些新的。

@JamieMaple
Copy link
Author

JamieMaple commented Sep 29, 2018

@SnowOnion 我是两本对照着看的👀 http://book.realworldhaskell.org/read/monad-transformers.html ,里面的 import 是没问题的,我觉得这个错误可能是在翻译时的错误,并非原文的错误

@JamieMaple
Copy link
Author

@SnowOnion 我是看完 learn you a haskell 过来的,我其实并没有觉得 RWH 老,配合原文的 comments 没什么很大的问题,相反还是能学到很多,另外主要是 haskell 资源较少,有什么推荐的么

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