html_url,issue_url,id,node_id,user,created_at,updated_at,author_association,body,reactions,performed_via_github_app,issue
https://github.com/pydata/xarray/pull/1936#issuecomment-368343446,https://api.github.com/repos/pydata/xarray/issues/1936,368343446,MDEyOklzc3VlQ29tbWVudDM2ODM0MzQ0Ng==,1217238,2018-02-25T20:51:36Z,2018-02-25T20:51:36Z,MEMBER,"> I also find it quite noisy and not very encouraging to submit PRs at an early ""quick n dirty"" stage, where you really shouldn't care about pep8...

I agree, but the alternative is to get a failed build error, and it can be a pain to dig through the Travis-CI to find out what went wrong. We use automated comments like this at work to enforce style, and on the whole I think they're probably a good thing.

I would certainly encourage contributors not to worry about these errors until they think their code is ready to submit.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,299601789
https://github.com/pydata/xarray/pull/1936#issuecomment-368301220,https://api.github.com/repos/pydata/xarray/issues/1936,368301220,MDEyOklzc3VlQ29tbWVudDM2ODMwMTIyMA==,10050469,2018-02-25T11:20:12Z,2018-02-25T11:20:12Z,MEMBER,"> I would like to disable the fixer (which appears to use autopep8) entirely

Yes, totally, stickler should NOT try to fix anything.

Would you like to turn off stickler entirely or just the auto-fixer? I'm fine with both: while I thinks it's nice to have a tool pointing out pep8 errors (I don't like to do it as a reviewer, it makes me feel like nit-picking), I also find it quite noisy and not very encouraging to submit PRs at an early ""quick n dirty"" stage, where you really shouldn't care about pep8...","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,299601789
https://github.com/pydata/xarray/pull/1936#issuecomment-368286733,https://api.github.com/repos/pydata/xarray/issues/1936,368286733,MDEyOklzc3VlQ29tbWVudDM2ODI4NjczMw==,1217238,2018-02-25T06:18:24Z,2018-02-25T06:18:24Z,MEMBER,"> Maybe we should turn off the fixer also?

I'm not sure if you were suggesting turning it off entirely or only for the docs... but as I wrote above, I don't like it either.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,299601789
https://github.com/pydata/xarray/pull/1936#issuecomment-368286655,https://api.github.com/repos/pydata/xarray/issues/1936,368286655,MDEyOklzc3VlQ29tbWVudDM2ODI4NjY1NQ==,1217238,2018-02-25T06:16:27Z,2018-02-25T06:16:27Z,MEMBER,"I would like to disable the fixer (which appears to use [autopep8](https://pypi.python.org/pypi/autopep8)) entirely, for several reasons:
1. It enforces a very strict style automatically, e.g., removing spaces around arithmetic operators. This actually goes beyond PEP8, which suggests ""Use your own judgment"". I would rather have comments that I can tell a contributor to ignore and easily disable in the future (via the flake8 config).
2. The need to pull changes to a pull-request branch from GitHub is a surprising and somewhat disruptive.
3. I can't figure out how to disable the fixer from only some files.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,299601789
https://github.com/pydata/xarray/pull/1936#issuecomment-368056843,https://api.github.com/repos/pydata/xarray/issues/1936,368056843,MDEyOklzc3VlQ29tbWVudDM2ODA1Njg0Mw==,10050469,2018-02-23T16:17:19Z,2018-02-23T16:17:19Z,MEMBER,Maybe we should turn off the fixer also?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,299601789