issue_comments

Table actions

1 row where issue = 288322322 and user = 306380 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

Suggested facets: created_at (date), updated_at (date)

user 1

  • mrocklin · 1 ✖

issue 1

  • Make `flake8 xarray` pass · 1 ✖

author_association 1

id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
357442522 https://github.com/pydata/xarray/pull/1824#issuecomment-357442522 https://api.github.com/repos/pydata/xarray/issues/1824 MDEyOklzc3VlQ29tbWVudDM1NzQ0MjUyMg== mrocklin 306380 2018-01-13T15:14:27Z 2018-01-13T15:14:27Z MEMBER

Huzzah. Thanks Zac for picking this up

On Sat, Jan 13, 2018 at 6:37 AM, Zac Hatfield Dodds notifications@github.com wrote:

Closes #1741 https://github.com/pydata/xarray/pull/1741 by @mrocklin https://github.com/mrocklin (who did most of the work I'm presenting here). I had an evening free, so I rebased the previous pull on master, fixed the conflicts, and then made everything pass with flake8's default settings (including line length). My condolences to whoever gets to review this diff!

The single change any non-pedant will notice: Travis now fails if there is a flake8 warning anywhere. My experience in other projects is that this is the only way to actually keep flake8 passing - it's just unrealistic to expect perfect attention to detail from every contributor, but "make the build green before we merge" is widely understood 😄

You can view, comment on, or merge this pull request online at:

https://github.com/pydata/xarray/pull/1824 Commit Summary

  • autoflake
  • autopep8
  • manual checking
  • flake8 entire codebase
  • fixup test
  • Xarray is now flake8-clean with dask settings
  • Fix all flake8 issues under default settings

File Changes

Patch Links:

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pydata/xarray/pull/1824, or mute the thread https://github.com/notifications/unsubscribe-auth/AASszFdoOfPL0XthNrYRAMofHkVvt9uPks5tKJWKgaJpZM4RdMeC . 

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Make `flake8 xarray` pass 288322322

Advanced export

JSON shape: default, array, newline-delimited, object

CSV options: 

CREATE TABLE [issue_comments] (
   [html_url] TEXT,
   [issue_url] TEXT,
   [id] INTEGER PRIMARY KEY,
   [node_id] TEXT,
   [user] INTEGER REFERENCES [users]([id]),
   [created_at] TEXT,
   [updated_at] TEXT,
   [author_association] TEXT,
   [body] TEXT,
   [reactions] TEXT,
   [performed_via_github_app] TEXT,
   [issue] INTEGER REFERENCES [issues]([id])
);
CREATE INDEX [idx_issue_comments_issue]
    ON [issue_comments] ([issue]);
CREATE INDEX [idx_issue_comments_user]
    ON [issue_comments] ([user]);