issue_comments
17 rows where "created_at" is on date 2022-09-05 sorted by id
This data as json, CSV (advanced)
Suggested facets: issue_url, author_association, updated_at (date)
id ▼ | html_url | issue_url | node_id | user | created_at | updated_at | author_association | body | reactions | issue | performed_via_github_app |
---|---|---|---|---|---|---|---|---|---|---|---|
1236511153 | https://github.com/simonw/sqlite-utils/issues/482#issuecomment-1236511153 | https://api.github.com/repos/simonw/sqlite-utils/issues/482 | IC_kwDOCGYnMM5Js6mx | simonw 9599 | 2022-09-05T03:38:29Z | 2022-09-05T03:38:29Z | OWNER | This is an interesting research area! Running the same experiment as the one in https://www.cybertec-postgresql.com/en/column-order-in-postgresql-does-matter/ could demonstrate one way or another if column ordering makes a difference with regards to performance or not. My hunch is that even if it does this wouldn't be a feature to bake into `sqlite-utils` itself, beyond the existing features for changing the order of columns in a table (https://sqlite-utils.datasette.io/en/stable/python-api.html#changing-column-order). | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | balanced table default column_order 1361355564 | |
1237374713 | https://github.com/simonw/datasette/issues/1800#issuecomment-1237374713 | https://api.github.com/repos/simonw/datasette/issues/1800 | IC_kwDOBm6k_c5JwNb5 | simonw 9599 | 2022-09-05T18:24:00Z | 2022-09-05T18:24:00Z | OWNER | I'll do this in a PR to execute the tests before merging to main. | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Remove upper bound dependencies as a default policy 1362363685 | |
1237375000 | https://github.com/simonw/datasette/issues/1800#issuecomment-1237375000 | https://api.github.com/repos/simonw/datasette/issues/1800 | IC_kwDOBm6k_c5JwNgY | simonw 9599 | 2022-09-05T18:24:34Z | 2022-09-05T18:24:34Z | OWNER | I'm going to leave `black` pinned to the exact version simply because updates to Black can cause my tests to fail due to formatting style decisions they have made - and it's just a test dependency, not a install requires dependency. | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Remove upper bound dependencies as a default policy 1362363685 | |
1237380827 | https://github.com/simonw/datasette/pull/1801#issuecomment-1237380827 | https://api.github.com/repos/simonw/datasette/issues/1801 | IC_kwDOBm6k_c5JwO7b | codecov[bot] 22429695 | 2022-09-05T18:35:24Z | 2022-09-05T18:35:24Z | NONE | # [Codecov](https://codecov.io/gh/simonw/datasette/pull/1801?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Simon+Willison) Report Base: **91.66**% // Head: **91.66**% // No change to project coverage :thumbsup: > Coverage data is based on head [(`3f24edd`)](https://codecov.io/gh/simonw/datasette/pull/1801?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Simon+Willison) compared to base [(`ba35105`)](https://codecov.io/gh/simonw/datasette/commit/ba35105eee2d3ba620e4f230028a02b2e2571df2?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Simon+Willison). > Patch has no changes to coverable lines. <details><summary>Additional details and impacted files</summary> ```diff @@ Coverage Diff @@ ## main #1801 +/- ## ======================================= Coverage 91.66% 91.66% ======================================= Files 38 38 Lines 4705 4705 ======================================= Hits 4313 4313 Misses 392 392 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Simon+Willison). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Simon+Willison) </details> [:umbrella: View full report at Codecov](https://codecov.io/gh/simonw/datasette/pull/1801?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Simon+Willison). :loudspeaker: Do you have feedback about the report comment? [Let us know in this issue](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&… | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Don't use upper bound dependencies, refs #1800 1362367821 | |
1237381569 | https://github.com/simonw/datasette/pull/1799#issuecomment-1237381569 | https://api.github.com/repos/simonw/datasette/issues/1799 | IC_kwDOBm6k_c5JwPHB | dependabot[bot] 49699333 | 2022-09-05T18:36:42Z | 2022-09-05T18:36:42Z | CONTRIBUTOR | Looks like aiofiles is no longer updatable, so this is no longer needed. | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Update aiofiles requirement from <0.9,>=0.4 to >=0.4,<22.2 1362242558 | |
1237381620 | https://github.com/simonw/datasette/pull/1685#issuecomment-1237381620 | https://api.github.com/repos/simonw/datasette/issues/1685 | IC_kwDOBm6k_c5JwPH0 | dependabot[bot] 49699333 | 2022-09-05T18:36:47Z | 2022-09-05T18:36:47Z | CONTRIBUTOR | Looks like jinja2 is no longer updatable, so this is no longer needed. | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Update jinja2 requirement from <3.1.0,>=2.10.3 to >=2.10.3,<3.2.0 1180778860 | |
1237383202 | https://github.com/simonw/datasette/issues/1800#issuecomment-1237383202 | https://api.github.com/repos/simonw/datasette/issues/1800 | IC_kwDOBm6k_c5JwPgi | simonw 9599 | 2022-09-05T18:39:43Z | 2022-09-05T18:39:52Z | OWNER | Dependabot noticed: https://github.com/simonw/datasette/pull/1799#issuecomment-1237381569 > Looks like aiofiles is no longer updatable, so this is no longer needed. | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Remove upper bound dependencies as a default policy 1362363685 | |
1237405647 | https://github.com/simonw/datasette/issues/1802#issuecomment-1237405647 | https://api.github.com/repos/simonw/datasette/issues/1802 | IC_kwDOBm6k_c5JwU_P | simonw 9599 | 2022-09-05T19:21:41Z | 2022-09-05T19:21:41Z | OWNER | Step one: replicate this failure on my local machine. I'll probably use Docker for this. | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Tests reliably failing on Python 3.7 1362402998 | |
1237519153 | https://github.com/simonw/datasette/issues/1802#issuecomment-1237519153 | https://api.github.com/repos/simonw/datasette/issues/1802 | IC_kwDOBm6k_c5Jwwsx | simonw 9599 | 2022-09-05T23:17:03Z | 2022-09-05T23:17:03Z | OWNER | Some previous issues where I've run Python in Docker: - https://github.com/simonw/datasette/issues/1344#issuecomment-849805778 - https://github.com/simonw/datasette/issues/1647#issuecomment-1059819628 For this one I'm going to try this image: `python:3.7-buster` - from https://hub.docker.com/layers/library/python/3.7-buster/images/sha256-8a79c5ddabd9e3bceb7d274e515d85a3f5898bd983bb5761ec94186bc54ae968?context=explore ``` docker run -it -v `pwd`:/mnt \ python:3.7-buster \ bash -c "cd /mnt && pip install -e '.[test]' && pytest" ``` | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Tests reliably failing on Python 3.7 1362402998 | |
1237522407 | https://github.com/simonw/datasette/issues/1802#issuecomment-1237522407 | https://api.github.com/repos/simonw/datasette/issues/1802 | IC_kwDOBm6k_c5Jwxfn | simonw 9599 | 2022-09-05T23:25:53Z | 2022-09-05T23:25:53Z | OWNER | Turned that into a TIL: https://til.simonwillison.net/docker/pytest-docker | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Tests reliably failing on Python 3.7 1362402998 | |
1237522547 | https://github.com/simonw/datasette/issues/1802#issuecomment-1237522547 | https://api.github.com/repos/simonw/datasette/issues/1802 | IC_kwDOBm6k_c5Jwxhz | simonw 9599 | 2022-09-05T23:26:19Z | 2022-09-05T23:26:19Z | OWNER | Annoyingly those tests pass just fine on my machine using that Docker recipe. | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Tests reliably failing on Python 3.7 1362402998 | |
1237522793 | https://github.com/simonw/datasette/issues/1802#issuecomment-1237522793 | https://api.github.com/repos/simonw/datasette/issues/1802 | IC_kwDOBm6k_c5Jwxlp | simonw 9599 | 2022-09-05T23:27:08Z | 2022-09-05T23:27:08Z | OWNER | Here's how GitHub Actions runs them: https://github.com/simonw/datasette/blob/294ecd45f7801971dbeef383d0c5456ee95ab839/.github/workflows/test.yml#L27-L37 | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Tests reliably failing on Python 3.7 1362402998 | |
1237523178 | https://github.com/simonw/datasette/issues/1802#issuecomment-1237523178 | https://api.github.com/repos/simonw/datasette/issues/1802 | IC_kwDOBm6k_c5Jwxrq | simonw 9599 | 2022-09-05T23:28:11Z | 2022-09-05T23:28:11Z | OWNER | Trying this instead: ``` docker run -it -v `pwd`:/mnt \ python:3.7-buster \ bash -c "cd /mnt && pip install -e '.[test]' && pytest -n auto -m 'not serial' && pytest -m 'serial'" ``` | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Tests reliably failing on Python 3.7 1362402998 | |
1237527575 | https://github.com/simonw/datasette/issues/1802#issuecomment-1237527575 | https://api.github.com/repos/simonw/datasette/issues/1802 | IC_kwDOBm6k_c5JwywX | simonw 9599 | 2022-09-05T23:40:32Z | 2022-09-05T23:40:32Z | OWNER | That passed too: ``` ========================================================= test session starts ========================================================== platform linux -- Python 3.7.13, pytest-7.1.3, pluggy-1.0.0 SQLite: 3.27.2 rootdir: /mnt, configfile: pytest.ini plugins: asyncio-0.19.0, anyio-3.6.1, timeout-2.1.0, xdist-2.5.0, forked-1.4.0 asyncio: mode=strict gw0 [1024] / gw1 [1024] / gw2 [1024] / gw3 [1024] / gw4 [1024] / gw5 [1024] ................................................................................................................................ [ 12%] ................................................................................................................................ [ 25%] ...............................................................................................................ss............... [ 37%] ................................................................................................................................ [ 50%] ................................................................................................................................ [ 62%] .................................................s..........................s................................................... [ 75%] ................................................................................................................................ [ 87%] ................................................................................................................................ [100%] ============================================= 1020 passed, 4 skipped in 294.43s (0:04:54) ============================================== ========================================================= test session starts ========================================================== platform linux -- Python 3.7.13, pytest-7.1.3, pluggy-1.0.0 SQLite: 3.27.2 rootdir: /mnt, configfile: pytest.ini plugins: asyncio-0.19.0, anyio-3.6.1, timeout-2.1.0, xdist-2.5.0, forked-1.4.0 asyncio: mode=strict … | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Tests reliably failing on Python 3.7 1362402998 | |
1237530726 | https://github.com/simonw/datasette/issues/1802#issuecomment-1237530726 | https://api.github.com/repos/simonw/datasette/issues/1802 | IC_kwDOBm6k_c5Jwzhm | simonw 9599 | 2022-09-05T23:50:18Z | 2022-09-05T23:50:18Z | OWNER | Tried again, this time it failed with 3.8: https://github.com/simonw/datasette/runs/8197344556?check_suite_focus=true I'm going to try ditching the parallel test execution. | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Tests reliably failing on Python 3.7 1362402998 | |
1237532571 | https://github.com/simonw/datasette/issues/1802#issuecomment-1237532571 | https://api.github.com/repos/simonw/datasette/issues/1802 | IC_kwDOBm6k_c5Jwz-b | simonw 9599 | 2022-09-05T23:56:28Z | 2022-09-05T23:56:28Z | OWNER | That didn't help. This time it was 3.9 that failed first: https://github.com/simonw/datasette/runs/8197403744?check_suite_focus=true - same error though. | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Tests reliably failing on Python 3.7 1362402998 | |
1237533471 | https://github.com/simonw/datasette/issues/1802#issuecomment-1237533471 | https://api.github.com/repos/simonw/datasette/issues/1802 | IC_kwDOBm6k_c5Jw0Mf | simonw 9599 | 2022-09-05T23:59:26Z | 2022-09-05T23:59:26Z | OWNER | https://docs.python.org/3/library/asyncio-task.html#asyncio.run notes that `asyncio.run()` was added in Python 3.7. I dropped support for 3.6 a while ago. https://stackoverflow.com/a/56446830/6083 suggests that switching to `asyncio.run()` has helped other people dealing with the `RuntimeError: There is no current event loop in thread` error in the past. | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Tests reliably failing on Python 3.7 1362402998 |
Advanced export
JSON shape: default, array, newline-delimited, object
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, [issue] INTEGER REFERENCES [issues]([id]) , [performed_via_github_app] TEXT); CREATE INDEX [idx_issue_comments_issue] ON [issue_comments] ([issue]); CREATE INDEX [idx_issue_comments_user] ON [issue_comments] ([user]);
created_at (date) 1 ✖