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

Course.Monad is a red herring #118

Closed
tonymorris opened this issue Feb 14, 2015 · 0 comments
Closed

Course.Monad is a red herring #118

tonymorris opened this issue Feb 14, 2015 · 0 comments

Comments

@tonymorris
Copy link
Contributor

People keep asking about Course.Monad and what exercises should be completed, however, there are no exercises in the source file. That's because Monad is a combination of the Applicative and Bind type-classes, where all the exercises are done. It is also because the README puts the module in the progression and after completing exercises in other modules, it is unclear what to do with Course.Monad.

The only real exercise is a thinking one. The understanding that the Monad type-class is the coming together of its sub type-classes (Applicative and Bind).

It is suggested that Course.Monad be removed from the suggested progression and that the module itself contain a comment clarifying its purpose.

tonymorris added a commit that referenced this issue Feb 14, 2015
feynmanliang added a commit to feynmanliang/fp-course that referenced this issue Jun 15, 2017
The reference to system-f#118 was because `Course.Monad` used to lack exercises since  [Course.Bind contained all of them](system-f@d63e288#diff-0a369498a5a8db3ac8fa606b544c9810R186). However, this was addressed in system-f#164 and system-f@9414bf2#diff-93667008909f8bff80964d12b2d3473b.

This PR updates the readme to remove the deprecated remark.
ajmcmiddlin pushed a commit to ajmcmiddlin/fp-course that referenced this issue Jan 18, 2018
The reference to system-f#118 was because `Course.Monad` used to lack exercises since  [Course.Bind contained all of them](system-f@d63e288#diff-0a369498a5a8db3ac8fa606b544c9810R186). However, this was addressed in system-f#164 and system-f@9414bf2#diff-93667008909f8bff80964d12b2d3473b.

This PR updates the readme to remove the deprecated remark.
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

No branches or pull requests

1 participant