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
Fix uk locale abbreviated months matcher #3139
Fix uk locale abbreviated months matcher #3139
Conversation
@leshakoss Is there any way to speed up reviewing and fixing the locale issues ? There are many issues raised on several locales due to format -> parse tokens. Issues that we would like to resolve sooner than later, for our next release that's on hold because of them. I'm trying to help as I can, with your guidance of course, but I don't see much activity on them. I asked my manager to donate as well for this library as it is a core dependency of our product. |
@vadimpopa hi, could you please change the base branch for this PR from |
ec38648
to
d01fe58
Compare
@leshakoss all good now ? |
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.
Yep, thanks a lot!
@leshakoss I see some failing checks due to a snapshot conflict. Anything to do about it ?
|
Released this as v2.29.2: https://github.com/date-fns/date-fns/releases/tag/v2.29.2 Thanks for your contribution! |
<p>This PR was automatically created by Snyk using the credentials of a real user.</p><br /><h3>Snyk has created this PR to upgrade date-fns from 2.29.2 to 2.29.3.</h3> :information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project. <hr/> - The recommended version is **1 version** ahead of your current version. - The recommended version was released **6 months ago**, on 2022-09-13. <details> <summary><b>Release notes</b></summary> <br/> <details> <summary>Package name: <b>date-fns</b></summary> <ul> <li> <b>2.29.3</b> - <a href="https://snyk.io/redirect/github/date-fns/date-fns/releases/tag/v2.29.3">2022-09-13</a></br><p>This release is prepared by our own <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/leshakoss/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://snyk.io/redirect/github/leshakoss">@ leshakoss</a>.</p> <h3>Fixed</h3> <ul> <li> <p><a href="https://snyk.io/redirect/github/date-fns/date-fns/pull/3175" data-hovercard-type="pull_request" data-hovercard-url="/date-fns/date-fns/pull/3175/hovercard">Fixed Ukrainian (<code>uk</code>) locale grammar for <code>formatDistance</code>.</a></p> </li> <li> <p><a href="https://snyk.io/redirect/github/date-fns/date-fns/pull/3167" data-hovercard-type="pull_request" data-hovercard-url="/date-fns/date-fns/pull/3167/hovercard">Improved browser compatibility by transforming the code with <code>@ babel/preset-env</code>.</a></p> </li> </ul> </li> <li> <b>2.29.2</b> - <a href="https://snyk.io/redirect/github/date-fns/date-fns/releases/tag/v2.29.2">2022-08-18</a></br><p>This release is brought to you by <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/nopears/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://snyk.io/redirect/github/nopears">@ nopears</a>, <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/vadimpopa/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://snyk.io/redirect/github/vadimpopa">@ vadimpopa</a> and <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/leshakoss/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://snyk.io/redirect/github/leshakoss">@ leshakoss</a>.</p> <h3>Fixed</h3> <ul> <li> <p><a href="https://snyk.io/redirect/github/date-fns/date-fns/pull/3160" data-hovercard-type="pull_request" data-hovercard-url="/date-fns/date-fns/pull/3160/hovercard">Fixed <code>sv</code> locale abbreviated months matcher.</a></p> </li> <li> <p><a href="https://snyk.io/redirect/github/date-fns/date-fns/pull/3139" data-hovercard-type="pull_request" data-hovercard-url="/date-fns/date-fns/pull/3139/hovercard">Fixed <code>uk</code> locale abbreviated months matcher.</a></p> </li> <li> <p><a href="https://snyk.io/redirect/github/date-fns/date-fns/pull/3153" data-hovercard-type="pull_request" data-hovercard-url="/date-fns/date-fns/pull/3153/hovercard">Fixed a breaking change in <code>intervalToDuration</code> by removing a recently introduced RangeError.</a></p> </li> </ul> </li> </ul> from <a href="https://snyk.io/redirect/github/date-fns/date-fns/releases">date-fns GitHub release notes</a> </details> </details> <hr/> **Note:** *You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.* For more information: <img src="https://api.segment.io/v1/pixel/track?data=eyJ3cml0ZUtleSI6InJyWmxZcEdHY2RyTHZsb0lYd0dUcVg4WkFRTnNCOUEwIiwiYW5vbnltb3VzSWQiOiI2ZjdlODRkMS0zNTZmLTQzY2MtYjYzYy05MzEyZWYyZDcwMjAiLCJldmVudCI6IlBSIHZpZXdlZCIsInByb3BlcnRpZXMiOnsicHJJZCI6IjZmN2U4NGQxLTM1NmYtNDNjYy1iNjNjLTkzMTJlZjJkNzAyMCJ9fQ==" width="0" height="0"/> 🧐 [View latest project report](https://app.snyk.io/org/mergestat/project/e267d993-52da-4f6e-9de9-0012cd8b1835?utm_source=github&utm_medium=referral&page=upgrade-pr) 🛠 [Adjust upgrade PR settings](https://app.snyk.io/org/mergestat/project/e267d993-52da-4f6e-9de9-0012cd8b1835/settings/integration?utm_source=github&utm_medium=referral&page=upgrade-pr) 🔕 [Ignore this dependency or unsubscribe from future upgrade PRs](https://app.snyk.io/org/mergestat/project/e267d993-52da-4f6e-9de9-0012cd8b1835/settings/integration?pkg=date-fns&utm_source=github&utm_medium=referral&page=upgrade-pr#auto-dep-upgrades) <!--- (snyk:metadata:{"prId":"6f7e84d1-356f-43cc-b63c-9312ef2d7020","prPublicId":"6f7e84d1-356f-43cc-b63c-9312ef2d7020","dependencies":[{"name":"date-fns","from":"2.29.2","to":"2.29.3"}],"packageManager":"npm","type":"auto","projectUrl":"https://app.snyk.io/org/mergestat/project/e267d993-52da-4f6e-9de9-0012cd8b1835?utm_source=github&utm_medium=referral&page=upgrade-pr","projectPublicId":"e267d993-52da-4f6e-9de9-0012cd8b1835","env":"prod","prType":"upgrade","vulns":[],"issuesToFix":[],"upgrade":[],"upgradeInfo":{"versionsDiff":1,"publishedDate":"2022-09-13T12:36:12.348Z"},"templateVariants":[],"hasFixes":false,"isMajorUpgrade":false,"isBreakingChange":false,"priorityScoreList":[]}) --->
Fix uk locale abbreviated months matcher. Extend formatParserTokens testing with all year months.
The initial abbreviations were resulting many Invalid Dates and Errors, you can see here: 371638c#diff-e7a044747c760242f5fc578749d6b8ad313008832a49188cc711ce34b77ae5d9