home / github

Menu
  • GraphQL API
  • Search all tables

issue_comments

Table actions
  • GraphQL API for issue_comments

6 rows where author_association = "MEMBER" and issue = 1086346755 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

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

user 4

  • mathause 3
  • jhamman 1
  • max-sixty 1
  • keewis 1

issue 1

  • Replace distutils.version with packaging.version · 6 ✖

author_association 1

  • MEMBER · 6 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
1028303595 https://github.com/pydata/xarray/pull/6096#issuecomment-1028303595 https://api.github.com/repos/pydata/xarray/issues/6096 IC_kwDOAMm_X849Sqrr keewis 14808389 2022-02-02T19:54:40Z 2022-02-02T19:54:40Z MEMBER

and we don't need packaging to build the package itself (e.g. the wheel), which is why we don't list it in pyproject.toml

{
    "total_count": 3,
    "+1": 3,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Replace distutils.version with packaging.version 1086346755
1028301979 https://github.com/pydata/xarray/pull/6096#issuecomment-1028301979 https://api.github.com/repos/pydata/xarray/issues/6096 IC_kwDOAMm_X849SqSb jhamman 2443309 2022-02-02T19:52:31Z 2022-02-02T19:52:31Z MEMBER

@ax3l - The packaging dependency should be resolved in https://github.com/pydata/xarray/pull/6207.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Replace distutils.version with packaging.version 1086346755
1000865482 https://github.com/pydata/xarray/pull/6096#issuecomment-1000865482 https://api.github.com/repos/pydata/xarray/issues/6096 IC_kwDOAMm_X847p_7K mathause 10194086 2021-12-24T14:51:01Z 2021-12-24T14:51:01Z MEMBER

Thanks for the feedback!

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Replace distutils.version with packaging.version 1086346755
1000384550 https://github.com/pydata/xarray/pull/6096#issuecomment-1000384550 https://api.github.com/repos/pydata/xarray/issues/6096 IC_kwDOAMm_X847oKgm mathause 10194086 2021-12-23T15:41:46Z 2021-12-23T15:41:46Z MEMBER

:+1: thanks! I'll update the PR.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Replace distutils.version with packaging.version 1086346755
1000373371 https://github.com/pydata/xarray/pull/6096#issuecomment-1000373371 https://api.github.com/repos/pydata/xarray/issues/6096 IC_kwDOAMm_X847oHx7 mathause 10194086 2021-12-23T15:22:12Z 2021-12-23T15:22:12Z MEMBER

Note that packaging.version.parse will soon be effectively the same as calling packaging.version.Version

Actually, I would prefer Version(xr.__version__) > Version("0.20.1") over version.parse(xr.__version__) > version.parse("0.20.1") (mostly because I think it looks better...). However, it doesn't seem that this PR will be merged soon, though... Do you know if there any good reason for us to use parse over Version?


And I should still figure out if DuckArrayModule("pint").duck_array_version is used/ public.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Replace distutils.version with packaging.version 1086346755
999204074 https://github.com/pydata/xarray/pull/6096#issuecomment-999204074 https://api.github.com/repos/pydata/xarray/issues/6096 IC_kwDOAMm_X847jqTq max-sixty 5635139 2021-12-22T01:10:16Z 2021-12-22T01:10:16Z MEMBER

Thanks @mathause !

The current version.parse looks good (but the others are fine if you prefer ofc)

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Replace distutils.version with packaging.version 1086346755

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]);
Powered by Datasette · Queries took 13.277ms · About: xarray-datasette