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 easter_sunday as holiday in Brandenburg (Ger) #143

Merged
merged 3 commits into from Jul 6, 2016
Merged

add easter_sunday as holiday in Brandenburg (Ger) #143

merged 3 commits into from Jul 6, 2016

Conversation

uvchik
Copy link
Contributor

@uvchik uvchik commented Jul 6, 2016

Sometimes there is a difference between Sunday and holiday (e.g. extra payments for workers). So Easter Sunday is a normal Sunday almost everywhere in Germany but in Brandenburg it is an holiday.

Thanks for this package :-)

Sometimes there is a difference between Sunday and holiday.
(e.g. extra payments for workers)
@brunobord
Copy link
Member

hello there, thanks a lot for this contribution. Although, as you may have seen in the "CONTRIBUTING" document, you will have to add a test to reflect these changes in the Brandeburg calendar. It shouldn't be that hard, you'll only have to make sure that the easter sunday is in the holiday list for this calendar for a couple of years.
(and don't forget to check the flake8 coding style conventions too)

You can run the tests locally like this (using Python 3.4 as an example):

tox -re py34,flake8

@uvchik
Copy link
Contributor Author

uvchik commented Jul 6, 2016

Sorry. Done!

@brunobord brunobord merged commit b2306e1 into workalendar:master Jul 6, 2016
@brunobord
Copy link
Member

it's okay, don't worry ; thanks a lot for your patch, it's greatly appreciated.

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