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/issues/2627#issuecomment-450553450,https://api.github.com/repos/pydata/xarray/issues/2627,450553450,MDEyOklzc3VlQ29tbWVudDQ1MDU1MzQ1MA==,6815844,2018-12-30T11:08:41Z,2018-12-30T11:08:41Z,MEMBER,Closed via #2632 ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,393710539
https://github.com/pydata/xarray/issues/2627#issuecomment-449783531,https://api.github.com/repos/pydata/xarray/issues/2627,449783531,MDEyOklzc3VlQ29tbWVudDQ0OTc4MzUzMQ==,1217238,2018-12-25T01:23:04Z,2018-12-25T01:23:04Z,MEMBER,"Let's add a build check of some sort for flake8, maybe to use alongside pep8speaks?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,393710539
https://github.com/pydata/xarray/issues/2627#issuecomment-449774797,https://api.github.com/repos/pydata/xarray/issues/2627,449774797,MDEyOklzc3VlQ29tbWVudDQ0OTc3NDc5Nw==,5635139,2018-12-24T22:22:04Z,2018-12-24T22:22:04Z,MEMBER,Down a level: `# flake8: noqa` skips a whole file when placed on its own line. `# noqa` skips a line when placed at the end of the line. I saw a few of the former attempting to do the latter,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,393710539
https://github.com/pydata/xarray/issues/2627#issuecomment-449774747,https://api.github.com/repos/pydata/xarray/issues/2627,449774747,MDEyOklzc3VlQ29tbWVudDQ0OTc3NDc0Nw==,5635139,2018-12-24T22:21:04Z,2018-12-24T22:21:04Z,MEMBER,"> s strict pep8 compliance supposed to be mandatory in xarray? Sometimes it makes more sense to violate PEP8 to improve readability.
My vote is:
- Strict compliance isn't _that_ important, and secondary to readability
- Standards, supported by automated checks, ease collaborating and scaling
...so by-and-large do whatever you think is best, but make the checks pass with a comment where the code violates a standard","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,393710539
https://github.com/pydata/xarray/issues/2627#issuecomment-449749879,https://api.github.com/repos/pydata/xarray/issues/2627,449749879,MDEyOklzc3VlQ29tbWVudDQ0OTc0OTg3OQ==,1217238,2018-12-24T16:02:49Z,2018-12-24T16:02:49Z,MEMBER,"We do aim for strict PEP8 compliance, but this is hard to enforce without the automated checks. Cases where it's violated intentionally can be marked with a comment.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,393710539
https://github.com/pydata/xarray/issues/2627#issuecomment-449738704,https://api.github.com/repos/pydata/xarray/issues/2627,449738704,MDEyOklzc3VlQ29tbWVudDQ0OTczODcwNA==,35968931,2018-12-24T14:11:13Z,2018-12-24T14:11:13Z,MEMBER,"@max-sixty Is strict pep8 compliance supposed to be mandatory in xarray? Sometimes it makes more sense to violate PEP8 to improve readability. My PR #2553 has some examples - e.g. [here](https://github.com/pydata/xarray/blob/c2ce5ea83b5924302653c8dfba7de68c7d98c942/xarray/tests/test_combine.py#L646) where I deliberately used a longer line rather than two lines so that it was clearer that I was testing what happens when you concatenate a square of 4 datasets. On the other hand there are also some violations of pep8 in the PR which are just oversights, and did get lost in the (lengthy) discussion.
Is strict pep8 compliance already supposed to be mandatory in xarray? Should any intentional pep8 violations be flagged using `# noqa` so that flake8 ignores that line? ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,393710539
https://github.com/pydata/xarray/issues/2627#issuecomment-449684619,https://api.github.com/repos/pydata/xarray/issues/2627,449684619,MDEyOklzc3VlQ29tbWVudDQ0OTY4NDYxOQ==,5635139,2018-12-24T04:26:59Z,2018-12-24T04:26:59Z,MEMBER,"OK great - I've PRed changes to get back to zero
It would be good if pep8speaks added a check too, but looks like it's not available. We could either:
- Add one to travis
- Look into Stickler? (It looks online and active, but potentially we had a bad experience last time?)
- No system changes, but reviewers are more attentive to the initial pep8speaks comment","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,393710539
https://github.com/pydata/xarray/issues/2627#issuecomment-449660767,https://api.github.com/repos/pydata/xarray/issues/2627,449660767,MDEyOklzc3VlQ29tbWVudDQ0OTY2MDc2Nw==,1217238,2018-12-23T20:12:27Z,2018-12-23T20:12:27Z,MEMBER,I think I need to be more careful to check the first comment for PEP8 issues before merging -- I've been relying on the green checkmarks.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,393710539
https://github.com/pydata/xarray/issues/2627#issuecomment-449619351,https://api.github.com/repos/pydata/xarray/issues/2627,449619351,MDEyOklzc3VlQ29tbWVudDQ0OTYxOTM1MQ==,6815844,2018-12-23T07:23:10Z,2018-12-23T07:23:10Z,MEMBER,"@max-sixty
Pep8speaks edits his first comment for every push (he looks testing) probably to avoid adding too many comments to the thread.
But it is now easy to review the pep8 violation if the thread becomes large.
Personally, I feel it is not the best solution for the smooth development. If you have an alternative in your mind and could set it up, it'll be super nice.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,393710539