razorpay / i18nify

One stop solution for all your internationalisation needs.
https://geosmart.razorpay.com/
21 stars 10 forks source link

[chore]: checkout action added in dorny path filter check #152

Closed tarun-khanna closed 5 months ago

tarun-khanna commented 5 months ago

Description

Dorny path filter was failing in CodeCove coverage workflow when the PR got merged to master. Failing github action https://github.com/razorpay/i18nify/actions/runs/9265600710/job/25487985207 This PR adds a fix for same.

Changes Made

List the main changes made in this pull request.

Title ---
JIRA link https://razorpay.atlassian.net/browse/ROW-144
Slack thread (if any) NA
Product spec NA
Tech spec/One-pager NA
Bundle Size Difference NA
Fixes Issue NA

Type of change

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Additional Notes

Any additional information that would be helpful for the reviewer.

Checklist:

Reviewer Checklist


PR Title Format

Format: <type>: <subject>

Types can be as follows:

changeset-bot[bot] commented 5 months ago

⚠️ No Changeset found

Latest commit: 1f59d900d2e19b926464fe81132fc03d45829682

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

github-actions[bot] commented 5 months ago

Unit Test Results

0 files  0 suites   0s :stopwatch: 0 tests 0 :heavy_check_mark: 0 :zzz: 0 :x:

Results for commit 1f59d900.

codecov-commenter commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.52%. Comparing base (1047227) to head (1f59d90). Report is 36 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #152 +/- ## ========================================== + Coverage 95.00% 96.52% +1.51% ========================================== Files 44 48 +4 Lines 501 662 +161 Branches 128 162 +34 ========================================== + Hits 476 639 +163 + Misses 24 23 -1 + Partials 1 0 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

rzpcibot commented 5 months ago
Fails
:no_entry_sign: 🕵 Whoops, I don't see any reviewers. Remember to add one.

Bundle Size Report

Files

click to expand/collapse
| 🟢 No Change | 🗑 File Deleted | 🆕 New File | 📈 Size Increased | 👍 Size Reduced | | --- | --- | --- | --- | --- |
Parsed (kb)
🚦 File Name Base PR Diff %
👍 cjs/index.js 146.23 146.22 $\textcolor{green}{-0.01}$ -0.01
🟢 esm/index.min.js 52.8 52.8 $\textcolor{green}{0}$ 0
🟢 umd/index.js 166.76 166.76 $\textcolor{green}{0}$ 0

Generated by :no_entry_sign: dangerJS against 1f59d900d2e19b926464fe81132fc03d45829682

github-actions[bot] commented 5 months ago

Unit Test Results

0 files  0 suites   0s :stopwatch: 0 tests 0 :heavy_check_mark: 0 :zzz: 0 :x:

Results for commit 4d52f92e.