JuliaLang/julia

Dates round/floor/ceil arguments switched

Aperta

#47.786 aperta il 2 dic 2022

Vedi su GitHubGuida contributor
 (7 commenti) (4 reazioni) (0 assegnatari)Julia (5773 fork)batch import
datesgood first issuehelp wanted

Metriche repository

Star
 (48.709 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

round(T, x) is the normal API, but in Dates.jl the arguments are reversed.

julia> ceil(today(), Week)
2022-12-05

julia> ceil(Week, today())
ERROR: MethodError: no method matching ceil(::Type{Week}, ::Date)

Guida contributor

Direzione di ricerca
Ispeziona il codice sorgente del modulo Dates per individuare le definizioni delle funzioni round, floor e ceil. Identifica l'ordine degli argomenti e scambiali per corrispondere al pattern API standard di func(x, T). Assicura la coerenza con altre funzioni simili.
Tech stack
Nessuno
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JuliaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75