Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BR/Black Awareness Day as National Holiday #489

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tiagoornelas
Copy link

Updates Black Awareness Day for BR since it is a national holiday beginning on 2024.

News about the law that changed to National Holiday: https://g1.globo.com/politica/noticia/2023/12/21/lula-sanciona-lei-que-torna-nacional-o-feriado-da-consciencia-negra-em-20-de-novembro.ghtml

Wikipedia: https://pt.wikipedia.org/wiki/Dia_Nacional_de_Zumbi_e_da_Consci%C3%AAncia_Negra

The Wikipedia article was also updated to highlight the change to national holiday:

Foi oficializado feriado nacional em 21 de dezembro de 2023.[3]

@tiagoornelas
Copy link
Author

Hello, @commenthol . How are you doing?

I hope the changes in this PR is aligned with the best practices of your project. It will be really useful to update this specific holiday into national holiday for the brazilian users.

I'll be waiting for your review. Wish you the best!

@commenthol
Copy link
Owner

Hi @tiagoornelas,
Thanks for the PR.
It looks that the fixtures are not up to date. Can you please run and check the changes?

npx mocha test/all.mocha.js --writetests --countries BR

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.

2 participants