-
Notifications
You must be signed in to change notification settings - Fork 473
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 1991-2022 Russian substituted holidays #1957
Add 1991-2022 Russian substituted holidays #1957
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #1957 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 188 188
Lines 11421 11422 +1
Branches 1794 1794
=========================================
+ Hits 11421 11422 +1 ☔ View full report in Codecov by Sentry. |
I don't think there's a way to reduce "new code duplication" for our test suite here without changing how |
@KJhellico Understandable, in any case I've expanded and updated Russian substituted holidays down to 1991, as well as adding in-lieu observance for 1991-2025 from the Wikipedia source you posted above. 🙏 |
Quality Gate passedIssues Measures |
Proposed change
Add 1991-2022 Russian substituted holidays, 1991-2025 Russian observed holidays.
Resolves #432 .
Type of change
Checklist
make pre-commit
, it didn't generate any changesmake test
, all tests passed locally