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

Support conditions with alternative calendar #85

Merged

Conversation

kylekatarnls
Copy link
Owner

Fix #84

@codecov
Copy link

codecov bot commented Sep 21, 2021

Codecov Report

Merging #85 (327c031) into master (92a60b4) will not change coverage.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##              master       #85   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity       368       370    +2     
===========================================
  Files            433       433           
  Lines           1504      1510    +6     
===========================================
+ Hits            1504      1510    +6     
Impacted Files Coverage Δ
...mixin/BusinessDay/Calculator/HolidayCalculator.php 100.00% <100.00%> (ø)

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@kylekatarnls kylekatarnls force-pushed the fix/issue-84-support-alt-calendar-with-conditions branch from 818e6ec to 7f6eddf Compare September 21, 2021 08:47
@kylekatarnls kylekatarnls force-pushed the fix/issue-84-support-alt-calendar-with-conditions branch from 7f6eddf to 327c031 Compare April 14, 2022 16:25
@ziming
Copy link
Contributor

ziming commented Oct 18, 2024

I'm curious, what's the blocker blocking this since 2021?

@kylekatarnls
Copy link
Owner Author

I have to rebase, IIRC, some tests were failing which means the implementation broke some other features.

@kylekatarnls kylekatarnls force-pushed the fix/issue-84-support-alt-calendar-with-conditions branch from 327c031 to 7417615 Compare October 18, 2024 09:29
@kylekatarnls kylekatarnls merged commit f1c3ea1 into master Oct 18, 2024
76 checks passed
@kylekatarnls kylekatarnls deleted the fix/issue-84-support-alt-calendar-with-conditions branch October 18, 2024 09:46
@kylekatarnls
Copy link
Owner Author

I see we still have a bunch of things here to consider the feature complete: #84 (comment)

But it's not stopping from having this partial support merged.

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.

Cannot get expression to work properly
2 participants