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

add europe day for Luxembourg #377

Merged
merged 4 commits into from Jun 27, 2019

Conversation

sim0nx
Copy link
Contributor

@sim0nx sim0nx commented Jun 26, 2019

Since this year there is a new national holiday in Luxembourg "Europe Day".

  • Tests with a significant number of years to be tested for your calendar.
  • Changelog amended with a mention describing your changes.

@brunobord
Copy link
Member

Hi! Thanks a lot for this contribution, much appreciated!

However, if it's "since this year", I guess that this pull-request is wrong.

You may want to test that, prior to 2019, this Europe Day isn't a holiday, and in 2019 and beyond, this day was integrated into the holidays.
You may want to override the get_fixed_holidays method and test the value of the parameter "year" to correctly implement this.

@sim0nx
Copy link
Contributor Author

sim0nx commented Jun 26, 2019

Hi! Thanks a lot for this contribution, much appreciated!

However, if it's "since this year", I guess that this pull-request is wrong.

You may want to test that, prior to 2019, this Europe Day isn't a holiday, and in 2019 and beyond, this day was integrated into the holidays.
You may want to override the get_fixed_holidays method and test the value of the parameter "year" to correctly implement this.

Good point, oops! :-)
I fixed that.

@brunobord
Copy link
Member

Looks excellent! I'll try to merge this and make a bugfix release as soon as I can, stay tuned!

@sim0nx
Copy link
Contributor Author

sim0nx commented Jun 27, 2019

Looks excellent! I'll try to merge this and make a bugfix release as soon as I can, stay tuned!

Great! Appreciate it!

@brunobord brunobord merged commit e8f3fc6 into workalendar:master Jun 27, 2019
@brunobord
Copy link
Member

version 5.1.1 released on PyPI. Enjoy! https://pypi.org/project/workalendar/5.1.1/

@sim0nx
Copy link
Contributor Author

sim0nx commented Jun 27, 2019

Thanks! :-)

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.

None yet

2 participants