Break apart Contents Abstraction
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza con GithubRepo.get_contents y la documentación enlazada de la GitHub Contents API; después, sigue sus llamadores para distinguir la intención de directorios y archivos. Define un comportamiento separado para directorios y archivos, incluido el raw-media fallback para archivos de entre 1MB y 100MB; el issue también requiere decidir si se deben admitir archivos de más de 100MB.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
We currently have GithubRepo.get_contents as an abstraction around GitHub's Contents API endpoint. However, this endpoint behaves differently depending on what you're asking it for, in particular:
- a directory
- a file
- a file whose size is between 1MB and 100MB
- a file whose size is above 100MB
The method tries to handle those all in one place and the flow suffers because of it.
Instead we should pick apart this abstraction into two, based on the caller's intent:
- a directory
- a file, with fallbacks for larger files
For instance, file content for files between 1MB and 100MB can be retrieved with the application/vnd.github.v3.raw custom media type (as the Accept header). Since we already want the metadata from the initial call (it's not provided with a raw call) this seems like a nice fallback. We can discuss whether we need to support retrieving file content which is >100MB.
- Lenguaje dominante
- Python
- Estrellas
- 0
- Forks
- 0
- Merge medio
- 2 d 20 h
- PR fusionados (30 d)
- 9
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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 opensafely-core/osgithub
-
dependency-management
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
opensafely-core/osgithub#289 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
opensafely-core/osgithub#269 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
opensafely-core/osgithub#242 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
opensafely-core/osgithub#236 ·
-
`httpretty` looks unmaintained Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
opensafely-core/osgithub#230 · 1 comentario ·
Todos los issues de opensafely-core/osgithub
Issues similares
-
agent-ready documentation needs-triage
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
-
documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 91/100
-
workflow-status page template still says reusable workflows are "triggered only by workflow_call:" Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
-
Add https://search.jeremyh.xyz/ Abiertoinstance instance add
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
searxng/searx-instances#939 · 1 comentario ·
-
area-deployment area-integrations triage:bot-seen
Dificultad 2/5 Medio día Aptitud para principiantes 86/100