Contributing docs: local test runs silently skip SpatiaLite and ext.c tests, and the setup steps assume SSH and uv
メンテナーはふだん 3 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 84/100
- issue の種類
- ドキュメント
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- python, sqlite
- 領域
- documentation, testing
調査の方向性
docs/contributing.rstを.github/workflows/test.ymlと併せて読み、コントリビューター向けのセットアップをCI環境に合わせてください。HTTPSでforkをクローンする場合の手順を更新し、uvのインストール手順へのリンクを追加し、issueに記載されたコマンドを使ってSpatiaLiteとext.cのテストをローカルで実行するためのセットアップを記述してください。この作業は、ページでこれらの前提条件が説明され、コントリビューターが、そうしなければこれらのテストがローカルでスキップされる理由を特定できる状態になれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Following docs/contributing.rst on a fresh macOS machine, I hit three gaps:
-
The clone command uses SSH (
git clone [email protected]:simonw/sqlite-utils). It worked for me because I already have an SSH key on GitHub, but it fails withPermission denied (publickey)for anyone without one, and contributors usually need to clone their fork rather than the main repository. -
Every command uses
uv run, but the page does not say uv must be installed or link to its install instructions. -
uv run pytestpasses locally while skipping tests that CI runs. On my machine: 1,494 passed, 19 skipped.pytest -rsshows 12 skips intests/test_gis.py("Could not find SpatiaLite extension") and 3 intests/test_cli.py("Requires compiled ext.c"). CI installslibsqlite3-mod-spatialiteand compilesext.cin.github/workflows/test.yml, but the contributing page mentions neither, so a contributor changing GIS code can get a green local run and a red CI run.
Environment: macOS 26.2, Python 3.11.5, uv 0.13.7, sqlite-utils main at 9c66baf.
Proposed fix, in docs/contributing.rst: clone your fork over HTTPS, link https://docs.astral.sh/uv/getting-started/installation/, and add a "Tests that skip locally" section with brew install spatialite-tools (macOS) or apt-get install libsqlite3-mod-spatialite (Ubuntu) and cd tests && gcc ext.c -fPIC -shared -o ext.so.
Related: #900 (docs build command) and #872 (CI legs that never run a pandas test). Happy to open a PR for this.
- 主要言語
- Python
- スター
- 2.2k
- フォーク
- 215
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
simonw/sqlite-utils のほかの issue
-
Tests use CliRunner.isolated_filesystem(), which is deprecated and will be removed in Click 9.0対応中かも @devpryan7792 が 11 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
simonw/sqlite-utils#916 ·
メンテナーはふだん 3 日以内に返信
-
rows_from_file() splits a single-column CSV file on a letter, or raises csv.Error対応中かも @L4XB が 20 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
simonw/sqlite-utils#907 ·
メンテナーはふだん 3 日以内に返信
-
enable_fts() exhausts iterator columns before populating the index対応中かも @syahra712 が 25 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
simonw/sqlite-utils#902 ·
メンテナーはふだん 3 日以内に返信
-
rows_from_file() ignores ignore_extras= and extras_key= for anything that isn't Format.CSV対応中かも @feiiiiii5 が 37 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
simonw/sqlite-utils#892 ·
メンテナーはふだん 3 日以内に返信
-
Raw numpy scalar values get silently stored as corrupted BLOBs instead of being converted対応中かも @TanbirRamim が 43 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
simonw/sqlite-utils#876 ·
メンテナーはふだん 3 日以内に返信
simonw/sqlite-utils の issue をすべて見る
似ている issue
-
area/test type/bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
Clodmony/starship-coop#70 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
punitarani/fli#270 ·
-
🐛 Bug 🔔 Pending processing
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
jumpserver/jumpserver#17980 ·
メンテナーはふだん 1 日以内に返信
-
community enhancement good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
メンテナーはふだん 1 日以内に返信
-
namespace
難易度 2/5 1〜3時間 初心者へのやさしさ 60/100
EclipseFdn/open-vsx.org#15341 ·
メンテナーはふだん 1 日以内に返信