Skip to content

feat: add function that returns holidays of the year#397

Closed
vicentevendramin wants to merge 1 commit into
brazilian-utils:mainfrom
vicentevendramin:feat/national-holidays
Closed

feat: add function that returns holidays of the year#397
vicentevendramin wants to merge 1 commit into
brazilian-utils:mainfrom
vicentevendramin:feat/national-holidays

Conversation

@vicentevendramin

Copy link
Copy Markdown
Contributor

Implementei uma função que retorna todos os feriados nacionais (é importante verificar se algum está faltando).

Criei um objeto para armazenar os feriados fixos, que são aqueles com datas específicas. Para os feriados móveis, utilizei o algoritmo de Computus para calcular a data da Páscoa, e, a partir dela, determinei as datas dos outros feriados que não possuem uma data fixa.

Se precisar de mais ajustes ou outro tom, é só avisar!

resolve #395

@hyanmandian

Copy link
Copy Markdown
Member

Fala @vicentevendramin tudo certinho? Cara, estou trabalhando num refactoring pra atualizar a lib como um todo, tu se importa se eu deixar esse PR aberto ate eu terminar esse refactoring? Assim que terminar, te aviso aqui (ou eu posso aplicar as mudancas) para se adequar ao novo formato, oq tu acha?

@hyanmandian

Copy link
Copy Markdown
Member

Incluirei como parte do refactoring #398, vou te incluir nos contributors assim que terminar. Sinta-se a vontade a fazer o code review!

@hyanmandian hyanmandian mentioned this pull request Jan 26, 2025
40 tasks
@vicentevendramin

Copy link
Copy Markdown
Contributor Author

E aí, obrigado pelo retorno @hyanmandian ! Acho muito válido, fico no aguardo do refactor da lib. Se precisar de uma ajudinha, pode contar comigo 🫡

@vicentevendramin

Copy link
Copy Markdown
Contributor Author

Fala @hyanmandian! Vi que o refactoring terminou, ficou show!

Queria saber se minha contribuição ajudou no final das contas? Se sim, você consegue me adicionar como contributor do projeto? Lembrei que você tinha mencionado isso antes do refactor e gostaria de ter essa participação registrada, iria me ajudar bastante

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Retornar todos os Feriados Nacionais de um Ano

2 participants