-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
expression: fix compatibility issue related to ADDDATE
#13827
Conversation
ADDDATE
/run-all-tests |
Codecov Report
@@ Coverage Diff @@
## master #13827 +/- ##
===========================================
Coverage 80.1536% 80.1536%
===========================================
Files 480 480
Lines 118954 118954
===========================================
Hits 95346 95346
Misses 16081 16081
Partials 7527 7527 |
@js00070 Thanks for your contribution! |
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.
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
/run-all-tests |
@js00070 merge failed. |
/run-cherry-picker |
Signed-off-by: ti-srebot <[email protected]>
cherry pick to release-3.0 in PR #21008 |
…1008) Signed-off-by: ti-srebot <[email protected]>
What problem does this PR solve?
try to fix issue #13822
What is changed and how it works?
modify the regular expression related to
DATE
string convertion.Check List
Tests