Skip to content

Print debugging info to stderr so we don't clobber html output#2

Merged
iphydf merged 2 commits intoTokTok:masterfrom
GrayHatter:master
Oct 26, 2016
Merged

Print debugging info to stderr so we don't clobber html output#2
iphydf merged 2 commits intoTokTok:masterfrom
GrayHatter:master

Conversation

@GrayHatter
Copy link
Contributor

No description provided.

@GrayHatter
Copy link
Contributor Author

This does do some stupid things to the output because it's threading the first http hit, but I'll leave that solution to someone who likes haskell

@iphydf
Copy link
Member

iphydf commented Oct 24, 2016

hPutStrLn is just a loop over the string's characters, and stderr is not line-buffered. We don't really need all this debug output. You can probably just delete all of it and write a single status line saying something like <!-- gathering data from github -->, regardless of the output. It doesn't bother me much to have xml comments in the text output. You could also just remove all of them. I had initially put the status output there because it was really slow. Now that everything is parallel, it only takes 4 seconds to produce a result, and the debug output is no longer really needed.

@CLAassistant
Copy link

CLAassistant commented Oct 26, 2016

CLA assistant check
All committers have signed the CLA.

@GrayHatter
Copy link
Contributor Author

@iphydf commented, I'm working on adding the ability to add statuses similar to travis/coveralls but as per standard, hs and I don't work well together.

@iphydf iphydf merged commit bf9b33a into TokTok:master Oct 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants