Break apart Contents Abstraction
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia con GithubRepo.get_contents e la documentazione collegata della GitHub Contents API, quindi traccia i chiamanti per distinguere l’intento relativo a directory e file. Definisci un comportamento separato per directory e file, incluso il raw-media fallback per i file tra 1MB e 100MB; l’issue richiede inoltre di decidere se i file oltre 100MB debbano essere supportati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Python
- Stelle
- 0
- Fork
- 0
- Merge medio
- 2g 20h
- PR unite (30g)
- 9
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di opensafely-core/osgithub
-
dependency-management
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
opensafely-core/osgithub#289 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
opensafely-core/osgithub#269 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 52/100
opensafely-core/osgithub#242 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
opensafely-core/osgithub#236 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 30/100
opensafely-core/osgithub#230 · 1 commento ·
Tutte le issue di opensafely-core/osgithub
Issue simili
-
agent-ready documentation needs-triage
Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 91/100
-
workflow-status page template still says reusable workflows are "triggered only by workflow_call:" Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
-
instance instance add
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
searxng/searx-instances#939 · 1 commento ·
-
area-deployment area-integrations triage:bot-seen
Difficoltà 2/5 Mezza giornata Idoneità per principianti 86/100