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

Added national holiday for BMF calendar (Black Awareness Day, Nov 20th) #321

Merged
merged 5 commits into from
Feb 6, 2024

Conversation

danilogalisteu
Copy link
Contributor

@danilogalisteu danilogalisteu commented Jan 15, 2024

Added new national holiday (Black Awareness Day) on November 20th starting in 2024. The same date was a municipal holiday until 2021.

The new holiday is officially added through "Circular Letter 215/2023", published on 2023-12-22 and available here.

Excerpt:
image

The latest official BMF calendar can be verified here.

I will add new tests next. Because of the old holiday on the same date, the previous test has to be modified with an end date.

…r 20th)

Added new national holiday (Black Awareness Day) on November 20th starting in 2024. The same date was previously a municipal holiday.
Added test for new holiday on Nov 20th starting in 2024
Fix old test that should check November 20th only up to 2023, before new holiday is active.
Fix test conditions
@rsheftel rsheftel merged commit 2eb992d into rsheftel:dev Feb 6, 2024
0 of 13 checks passed
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