bitwalker/timex

Most functions are incorrect for calendars other than Calendar.ISO

Open

#449 ouverte le 30 juil. 2018

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Elixir (406 forks)batch import
feature/enhancementhelp wantedlevel:advanced

Métriques du dépôt

Stars
 (1 807 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez le code source de Timex pour identifier quelles fonctions manipulent directement les structures de données d'Elixir et supposent le calendrier ISO. Concentrez vous sur `Timex.shift/2` et les fonctions similaires. Vérifiez la documentation pour voir s'il existe déjà des gardes ou des correspondances de motifs. Décidez s'il faut ajouter des implémentations prenant en compte le calendrier ou se limiter à l'ISO et mettre à jour la documentation en conséquence. Envisagez de consulter le module `Calendar` dans Elixir pour obtenir des conseils sur la prise en charge de plusieurs calendriers.
Stack technique
elixir
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ElixirTimex libraryCalendar module
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
35