Skip to content

Conversation

@mdimjasevic
Copy link
Contributor

@mdimjasevic mdimjasevic commented Jul 16, 2018

https://iohk.myjetbrains.com/youtrack/issue/TSD-77

Hoogle cannot generate documentation for the log-classifier project due to incorrect documentation formatting in two places. These need to be fixed so that Hoogle can be used with the project.

@mdimjasevic mdimjasevic requested a review from ksaric July 16, 2018 09:46
Copy link
Contributor

@ksaric ksaric left a comment

Choose a reason for hiding this comment

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

stack hoogle seems to work now, good job.
Just remove the second comment since it's confusing to leave it as such, while also it still reports the error which is making Hoogle unable to build:

src/Lib.hs:559:71: error:
    parse error on input ‘-- ^ ticketStatus == "new"’
    |
559 |     isTicketOpen TicketInfo{..} = tiStatus `elem` unsolvedTicketStatus-- ^ ticketStatus == "new"

src/Lib.hs Outdated

isTicketOpen :: TicketInfo -> Bool
isTicketOpen TicketInfo{..} = tiStatus `elem` unsolvedTicketStatus-- || ticketStatus == "new"
isTicketOpen TicketInfo{..} = tiStatus `elem` unsolvedTicketStatus-- ^ ticketStatus == "new"
Copy link
Contributor

Choose a reason for hiding this comment

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

You can drop this comment, this was left and we don't need it anymore.

@mdimjasevic mdimjasevic requested review from ksaric and robcohen July 16, 2018 10:06
@mdimjasevic
Copy link
Contributor Author

I am not sure as to how to make sure that Hoogle can properly generate documentation. I tried with rm -rf .stack-work/hoogle and then stack hoogle. Is that enough? @ksaric , your input would be appreciated.

Copy link
Contributor

@ksaric ksaric left a comment

Choose a reason for hiding this comment

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

LGTM.

@ksaric
Copy link
Contributor

ksaric commented Jul 17, 2018

@mdimjasevic Yes, as I already mentioned, that should be enough.

@mdimjasevic mdimjasevic merged commit b5f9163 into develop Jul 17, 2018
@mdimjasevic mdimjasevic deleted the tsd77-hoogle-fix branch July 17, 2018 09:43
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.

3 participants