bitwalker/timex

Most functions are incorrect for calendars other than Calendar.ISO

Open

#449 aberto em 30 de jul. de 2018

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)Elixir (406 forks)batch import
feature/enhancementhelp wantedlevel:advanced

Métricas do repositório

Stars
 (1.807 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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.

Guia do colaborador

Direção de pesquisa
Examine o código fonte do Timex para identificar quais funções manipulam diretamente estruturas de dados do Elixir e assumem o calendário ISO. Concentre se em `Timex.shift/2` e funções semelhantes. Verifique a documentação para ver se existem guardas ou correspondências de padrão existentes. Decida se deve adicionar implementações com suporte a múltiplos calendários ou restringir ao ISO e atualizar a documentação de acordo. Considere olhar o módulo `Calendar` no Elixir para orientação sobre como suportar múltiplos calendários.
Pilha de tecnologia
elixir
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
4
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
ElixirTimex libraryCalendar module
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
35