bitwalker/timex

Most functions are incorrect for calendars other than Calendar.ISO

Open

#449 aperta il 30 lug 2018

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Elixir (406 fork)batch import
feature/enhancementhelp wantedlevel:advanced

Metriche repository

Star
 (1807 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Most functions in Timex manipulate the Elixir data structures directly. In doing so, they assume a lot about the underlying calendar - in most cases the functions would just not work for anything other than Calendar.ISO. The prime example would be Timex.shift/2.

Timex should either amend those functions to be correct in case they are called with other calendars, or pattern match on the value and document the functions to only work with Calendar.ISO.

Guida contributor

Direzione di ricerca
Esamina il codice sorgente di Timex per identificare quali funzioni manipolano direttamente le strutture dati di Elixir e presuppongono il calendario ISO. Concentrati su `Timex.shift/2` e funzioni simili. Controlla la documentazione per vedere se esistono già guardie o pattern matching. Decidi se aggiungere implementazioni consapevoli del calendario o limitarsi all'ISO e aggiornare di conseguenza la documentazione. Considera di consultare il modulo `Calendar` in Elixir per una guida sul supporto di più calendari.
Tech stack
elixir
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
4
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ElixirTimex libraryCalendar module
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
35