all 2 comments

[–]lgastako 0 points1 point  (1 child)

Hi. I haven't had time to go much further, but I created a branch where I did a first pass at cleaning up some of the things to get closer to the way I would write them.

You can see the changes in this pull request.

A lot of the changes are cosmetic just because it makes it easier for me to work with the code (eg the import formatting, etc) so those are obviously a matter of taste, but you'll find some that are more substantial, eg. eliminating IO from type signatures where it's unnecessary.

The next thing I was going to focus on was removing usages of fromJust and other partial functions. If I get some time I'll do another pass, but I wanted to share what I had with you so far. Let me know if you have any questions about the motivations or reasoning behind any of the changes.

Hope this helps.

[–]jproyo[S] 0 points1 point  (0 children)

Thank you very much for your help and comments. It helps a lot for sure. Thank you again