Contributing docs: local test runs silently skip SpatiaLite and ext.c tests, and the setup steps assume SSH and uv
Les mainteneurs répondent en général sous 3 jours
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 84/100
- Type d'issue
- Documentation
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- python, sqlite
- Domaine
- documentation, testing
Piste de recherche
Lisez docs/contributing.rst avec .github/workflows/test.yml afin d’aligner la configuration destinée aux contributeurs sur l’environnement CI. Mettez à jour les instructions de clonage pour les clones HTTPS de forks, ajoutez un lien vers les instructions d’installation de uv et documentez la configuration locale des tests SpatiaLite et ext.c avec les commandes de l’issue. Le travail est terminé lorsque la page explique ces prérequis et que les contributeurs peuvent comprendre pourquoi ces tests risqueraient sinon d’être ignorés localement.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Python
- Étoiles
- 2.2k
- Forks
- 215
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de simonw/sqlite-utils
-
FTS detection treats table names as LIKE patternsPeut-être pris @IlyaKhalafi l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
simonw/sqlite-utils#930 ·
Les mainteneurs répondent en général sous 3 jours
-
Tests use CliRunner.isolated_filesystem(), which is deprecated and will be removed in Click 9.0Peut-être pris @devpryan7792 l’a pris il y a 12 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
simonw/sqlite-utils#916 ·
Les mainteneurs répondent en général sous 3 jours
-
rows_from_file() splits a single-column CSV file on a letter, or raises csv.ErrorPeut-être pris @L4XB l’a pris il y a 21 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
simonw/sqlite-utils#907 ·
Les mainteneurs répondent en général sous 3 jours
-
enable_fts() exhausts iterator columns before populating the indexPeut-être pris @syahra712 l’a pris il y a 26 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
simonw/sqlite-utils#902 ·
Les mainteneurs répondent en général sous 3 jours
-
rows_from_file() ignores ignore_extras= and extras_key= for anything that isn't Format.CSVPeut-être pris @feiiiiii5 l’a pris il y a 38 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
simonw/sqlite-utils#892 ·
Les mainteneurs répondent en général sous 3 jours
Toutes les issues de simonw/sqlite-utils
Issues similaires
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
micropython/micropython#19906 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
FinanceFlash/unvibecode#597 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
TomAnthony/YourOwnWeigh#21 ·
Les mainteneurs répondent en général sous 1 jour
-
[BUG]: Compliance rules never see Unity Catalog views as viewsPeut-être pris @wagnercsantos l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
databrickslabs/ontos#1348 ·
Les mainteneurs répondent en général sous 3 jours
-
Deprecated datetime.utcnow() is called on every log line (DeprecationWarning on Python 3.12+)Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 2 jours