Consider switching to `src` layout
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 25/100
Línea de trabajo
The package layout change from #2277 is the starting point, so compare the repository tree against the src layout pattern and check the packaging config in pyproject.toml or setup.cfg. Moving the package under src/ touches imports, test discovery and the documentation build, so run the test suite afterwards. Done means the package installs from src/, the tests pass, and the maintainers agree on the final layout.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
To my mind, src is a separate discussion, and doesn't need to be done in https://github.com/pvlib/pvlib-python/pull/2277, if at all.
I brought it up because it seems to me that the majority of projects that use
/testsalso use/src. Indeed most of the linked packages on the PyOpenSci reference use/pkg/testsrather than/pkg+/tests. Alsopytestdocs include examples for either/src+/testsOR/pkg/testsbut not/pkg+/tests. Maybe it's ok but I like to follow the more popular patterns unless I know why I am deviating from them.
Originally posted by @wholmgren in #2271
#2277 moved the package from a flat layout to a "halfway-src" layout. We should consider adopting the src layout entirely, as the python community seems broadly agreed that the src layout is beneficial.
- Lenguaje dominante
- Python
- Estrellas
- 1.7k
- Forks
- 1.2k
- Merge medio
- 3 d 18 h
- PR fusionados (30 d)
- 13
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 pvlib/pvlib-python
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
pvlib/pvlib-python#2903 · 2 comentarios · 2 reacciones ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
pvlib/pvlib-python#2902 ·
Los mantenedores suelen responder en 1 día
-
Add an example or documentation on using 'bhi' from ERA5Quizá libre de nuevo @anshurajbisoyi98-ctrl la tomó hace 14 días y no hay ningún pull request abierto. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
pvlib/pvlib-python#2880 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Use "long-wave" rather than "infra-red" to refer to thermal radiation.Quizá libre de nuevo @Rahi-padwal la tomó hace 252 días y no hay ningún pull request abierto. Abiertoapi easy
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
pvlib/pvlib-python#2653 · 6 comentarios ·
Los mantenedores suelen responder en 1 día
-
question
Dificultad 1/5 1-3 horas Aptitud para principiantes 72/100
pvlib/pvlib-python#2637 · 10 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de pvlib/pvlib-python
Issues similares
-
bug Size: XS
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
oposs/repo-infra#93 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 69/100
Graphify-Labs/graphify#4546 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
521xueweihan/HelloGitHub#4025 ·