Skip to content

chore: use node js native utils#2441

Merged
andrii-bodnar merged 5 commits intolingui:nextfrom
yslpn:chore/use-node-js-native-utils
Feb 9, 2026
Merged

chore: use node js native utils#2441
andrii-bodnar merged 5 commits intolingui:nextfrom
yslpn:chore/use-node-js-native-utils

Conversation

@yslpn
Copy link
Contributor

@yslpn yslpn commented Feb 8, 2026 •
edited
Loading

Description

This PR includes two internal CLI maintenance updates to reduce dependencies and use native Node.js APIs:

  1. Replace glob package usage with built-in node:fs glob
  2. Replace es-main checks with native import.meta.main

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Examples update

Checklist

  • I have read the CONTRIBUTING and CODE_OF_CONDUCT docs
  • I have added tests that prove my fix is effective or that my feature works
  • I have added the necessary documentation (if appropriate)

@vercel
Copy link

vercel bot commented Feb 8, 2026

@yslpn is attempting to deploy a commit to the Crowdin Team on Vercel.

A member of the Team first needs to authorize it.

@yslpn
Copy link
Contributor Author

yslpn commented Feb 8, 2026

@andrii-bodnar Hello.

I'm getting an error in CI lint-pr-title, but I don't have permission to change the title. This is very inconvenient for contributors, as they can't change the target and title of a PR. Can you look at the settings?

@timofei-iatsenko timofei-iatsenko changed the title Chore/use node js native utils chore: use node js native utils Feb 9, 2026
@andrii-bodnar
Copy link
Contributor

andrii-bodnar commented Feb 9, 2026

Hello @yslpn, I couldn't find a specific option in the repository settings that allows to control permissions for editing the title of a pull request. I think this is allowed by default for users with 'Write' permission and above.

@yslpn
Copy link
Contributor Author

yslpn commented Feb 9, 2026

Hello @yslpn, I couldn't find a specific option in the repository settings that allows to control permissions for editing the title of a pull request. I think this is allowed by default for users with 'Write' permission and above.

Have you changed any settings in the last few months? I remember encountering a PR title error before and changing the title without any problems.

@andrii-bodnar andrii-bodnar deleted the branch lingui:next February 9, 2026 12:27
@andrii-bodnar andrii-bodnar reopened this Feb 9, 2026
@andrii-bodnar andrii-bodnar changed the base branch from chore/node-22 to next February 9, 2026 12:30
@andrii-bodnar
Copy link
Contributor

andrii-bodnar commented Feb 9, 2026

@yslpn no, I haven't changed anything recently, it's strange. Could you please rebase the current branch? I just retargeted it to the next branch

@andrii-bodnar andrii-bodnar added the ↻ awaiting rebase Rebase is needed due to conflicts label Feb 9, 2026
@timofei-iatsenko
Merge branch 'next' into fork/yslpn/chore/use-node-js-native-utils
2778bf4 
# Conflicts:
#	.github/workflows/main-suite.yml
#	.github/workflows/release.yml
#	.github/workflows/version-bump.yml
#	packages/cli/src/lingui-extract-experimental.ts
@timofei-iatsenko timofei-iatsenko removed the ↻ awaiting rebase Rebase is needed due to conflicts label Feb 9, 2026
@timofei-iatsenko timofei-iatsenko marked this pull request as ready for review February 9, 2026 13:52
@codecov
Copy link

codecov bot commented Feb 9, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 60.00000% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.66%. Comparing base (dd43fb0) to head (2778bf4).
⚠️ Report is 303 commits behind head on next.

Files with missing lines Patch % Lines
packages/cli/src/lingui-extract.ts 20.00% 4 Missing ⚠️

❌ Your patch status has failed because the patch coverage (60.00%) is below the target coverage (70.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files 
@@             Coverage Diff             @@
##             next    #2441       +/-   ##
===========================================
+ Coverage   76.66%   88.66%   +12.00%     
===========================================
  Files          81      116       +35     
  Lines        2083     3283     +1200     
  Branches      532      968      +436     
===========================================
+ Hits         1597     2911     +1314     
+ Misses        375      334       -41     
+ Partials      111       38       -73

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@andrii-bodnar andrii-bodnar added this to the v6 milestone Feb 9, 2026
@andrii-bodnar andrii-bodnar merged commit 7f4eb8e into lingui:next Feb 9, 2026
7 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andrii-bodnar andrii-bodnar andrii-bodnar approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v6

Development

Successfully merging this pull request may close these issues.

3 participants

@yslpn @andrii-bodnar @timofei-iatsenko