bitwalker/timex

Most functions are incorrect for calendars other than Calendar.ISO

Open

#449 geöffnet am 30. Juli 2018

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Elixir (406 Forks)batch import
feature/enhancementhelp wantedlevel:advanced

Repository-Metriken

Stars
 (1.807 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuchen Sie den Timex Quellcode, um zu identifizieren, welche Funktionen direkt mit Elixir Datenstrukturen arbeiten und den ISO Kalender voraussetzen. Konzentrieren Sie sich auf `Timex.shift/2` und ähnliche Funktionen. Überprüfen Sie die Dokumentation, um festzustellen, ob bereits vorhandene Guards oder Pattern Matches existieren. Entscheiden Sie, ob kalenderbewusste Implementierungen hinzugefügt oder auf ISO beschränkt und die Dokumentation entsprechend aktualisiert werden soll. Ziehen Sie in Betracht, das `Calendar` Modul in Elixir für Anleitungen zur Unterstützung mehrerer Kalender zu konsultieren.
Tech Stack
elixir
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
4
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
ElixirTimex libraryCalendar module
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
35