Proposal: an optional evidence check on pull requests (demo on 5 merged PRs)
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 12/100
Línea de trabajo
This is a proposal for an external evidence-check GitHub app and a possible comment-only CI workflow, not a code change with named files or tests in this repository. A newcomer would first need a maintainer decision on whether the project wants such a check at all. Done means that decision is recorded, and any follow-up work would be scoped in a separate issue.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hi @thodson-usgs, thanks again for the review and merge on #440.
I've been building proofcheck (https://github.com/bhaskargurram-ai/proofcheck). It's a GitHub check that tests a PR by running it. It runs the changed tests on the base and head commits, then reverts each changed hunk to see which ones the passing tests depend on.
To see whether it's useful here, I replayed five recently merged dataretrieval fixes in a fork. All five come back ✅:
| Upstream PR | Fail before the fix, pass after | Code hunks the tests depend on |
|---|---|---|
| #427 qualifier filter as JSON array | 3 tests | 4 of 4 |
| #429 numeric columns stay float64 | 7 tests | 4 of 4 |
| #431 reject unusable get_nearest_continuous inputs | 25 tests | 3 of 4 |
| #395 keep censored peaks | 9 tests | 1 of 1 |
| #432 Virgin Islands naming | 2 tests | 3 of 3 |
Cards: https://github.com/bhaskargurram-ai/dataretrieval-python/pulls
Docstring-only edits are marked separately and not counted. One honest caveat: my first run of #395 used the squash commit message as the description, and a per-commit note in it ("no behavior change") was misread as a claim about the whole PR. The real description says the opposite, so I re-ran it without that text.
If useful, I can open a PR adding it as a comment-only workflow that never fails CI. If not, feel free to close this.
- Lenguaje dominante
- Python
- Estrellas
- 269
- Forks
- 67
- Merge medio
- 3 d 17 h
- PR fusionados (30 d)
- 18
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de DOI-USGS/dataretrieval-python
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
DOI-USGS/dataretrieval-python#406 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Check getter parameter docs against the service's queryablesPosiblemente ocupada @acalebratliff la tomó hace 4 días. Abiertodocumentation ready-for-agent
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
DOI-USGS/dataretrieval-python#379 · 4 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 45/100
DOI-USGS/dataretrieval-python#260 ·
Los mantenedores suelen responder en 1 día
Todos los issues de DOI-USGS/dataretrieval-python
Issues similares
-
get_page_thumbnail modifies cached page images, affecting later thumbnails and picture exportsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
docling-project/docling-mcp#147 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
autowarefoundation/autoware_carla_scenario#405 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
mohitmishra786/readprism#322 ·
Los mantenedores suelen responder en 1 día