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 type informal to 'Dodenherdenking' [dutch] #126

Merged
merged 2 commits into from Jun 24, 2015
Merged

Add type informal to 'Dodenherdenking' [dutch] #126

merged 2 commits into from Jun 24, 2015

Conversation

MathijsK93
Copy link
Contributor

Dodenherdenking is in the netherlands not an official holiday.

More info (dutch):
http://www.rijksoverheid.nl/onderwerpen/schoolvakanties/vraag-en-antwoord/wanneer-zijn-de-officiele-feestdagen-in-nederland.html

Fixes #116

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.6%) to 83.29% when pulling d28886f on MathijsK93:master into 09997b6 on alexdunae:master.

@ppeble
Copy link
Member

ppeble commented Jun 16, 2015

Sorry! I gotta fix some stuff on travis and coveralls and then I'll get to this.

@ppeble
Copy link
Member

ppeble commented Jun 24, 2015

This looks good. I'll just get coveralls fixed and merge.

@ppeble
Copy link
Member

ppeble commented Jun 24, 2015

Actually, I am just going to merge this. The coveralls thing is unrelated. Thanks for the fix!

ppeble added a commit that referenced this pull request Jun 24, 2015
Add type informal to 'Dodenherdenking' [dutch]
@ppeble ppeble merged commit a31237f into holidays:master Jun 24, 2015
@MathijsK93
Copy link
Contributor Author

Thanks for the merge @ptrimble! When will the version be bumped?

@ppeble
Copy link
Member

ppeble commented Jun 25, 2015

I'll do it this morning. I meant to do it last night after I got off of the train for my commute but...I forgot. 😨

@MathijsK93
Copy link
Contributor Author

👍

@ppeble
Copy link
Member

ppeble commented Jun 25, 2015

Done! Let me know if you run into any issues.

Murphydbuffalo pushed a commit to Murphydbuffalo/holidays that referenced this pull request Sep 21, 2021
Murphydbuffalo pushed a commit to Murphydbuffalo/holidays that referenced this pull request Sep 21, 2021
implement missing holidays for ro region holidays#126
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.

How to override holidays
3 participants