-
-
Notifications
You must be signed in to change notification settings - Fork 561
Update CH holidays: rename Neujahrestag to Neujahrstag #2121
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
Conversation
Like in other German speaking countries, January 01 is called "Neujahr" in Switzerland, not "Neujahrestag". "Neujahr*e*stag" would be wrong anyway, it would be "Neujahrstag".
|
Here is "Neujahrstag". This is an official source, as far as I understand. |
That's correct, I'm familiar with that document. It is a document which specifies which days are counted as holidays for legal matters, for example to calculate whether you filed claims within the specified period of time. short answer: long answer: |
|
Yes, I mean, let's change it to "Neujahrstag". |
|
@kurtkeller does "Neujahrstag" look like a meet in the middle option? |
|
Yes, "Neujahrstag" is a good enough compromise. |
After discussions settled on removing the typo and use "Neujahrstag".
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #2121 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 192 192
Lines 11590 11599 +9
Branches 1747 1749 +2
=========================================
+ Hits 11590 11599 +9 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for spotting and fixing that @kurtkeller!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Co-authored-by: Arkadii Yakovets <ark@cho.red>
Proposed change
Like in other German speaking countries, January 01 is called "Neujahr" in Switzerland, not "Neujahrestag". "Neujahrestag" would be wrong anyway, it would be "Neujahrstag".
Type of change
holidaysfunctionality in general)Checklist
make check, all checks and tests are green