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

[Backport 1.16] fix(builtin-function): fix string conversion of years-months-durations #644

Merged
merged 2 commits into from
May 26, 2023

Conversation

saig0
Copy link
Member

@saig0 saig0 commented May 25, 2023

Description

Backport of #636 to 1.16.

Related issues

relates to #631

saig0 and others added 2 commits May 25, 2023 12:24
Clirr reports a false positive if an inner method is changed. Adjust the configuration of the Clirr plugin to ignore these errors.

(cherry picked from commit f9fde28)
* add tests for string conversion of zero-length years-months-duration
* add tests for string conversion of negative years-months-duration

(cherry picked from commit 38c8e38)
@saig0 saig0 requested a review from korthout May 25, 2023 10:35
@saig0
Copy link
Member Author

saig0 commented May 25, 2023

@korthout I need you to approve the PR. It's just a backport of a merged PR but the bot did make the job. 😄

Copy link
Member

@korthout korthout left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Without looking LGTM ✌️

@github-actions
Copy link

Successfully created backport PR for 1.15:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants