Skip to content

chore: partial migration to ESM#5932

Open
mostafaNazari702 wants to merge 1 commit into
mochajs:mainfrom
mostafaNazari702:esm-migration-phase-4
Open

chore: partial migration to ESM#5932
mostafaNazari702 wants to merge 1 commit into
mochajs:mainfrom
mostafaNazari702:esm-migration-phase-4

Conversation

@mostafaNazari702
Copy link
Copy Markdown
Contributor

@mostafaNazari702 mostafaNazari702 commented May 2, 2026

PR Checklist

Overview

I will push commits one by one and request individual reviews for each. first, I considered creating separate PRs for different phases, expecting a large number of modified files. However, not counting test files, we’re now close to a full migration. What remains are a few high-risk and sensitive areas where I’d appreciate your guidance on how to proceed.

@mark-wiemer @JoshuaKGoldberg

@codecov
Copy link
Copy Markdown

codecov Bot commented May 2, 2026

Codecov Report

❌ Patch coverage is 58.33333% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.73%. Comparing base (54dcfda) to head (4e21670).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
lib/cli/run.mjs 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5932      +/-   ##
==========================================
- Coverage   80.85%   79.73%   -1.13%     
==========================================
  Files          64       64              
  Lines        4566     4555      -11     
  Branches      958      979      +21     
==========================================
- Hits         3692     3632      -60     
- Misses        874      923      +49     

☔ 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.

@mostafaNazari702 mostafaNazari702 changed the title chore: migrate nodejs/serializer to ESM chore: migrate to ESM May 2, 2026
@mark-wiemer mark-wiemer self-requested a review May 2, 2026 21:01
@mostafaNazari702 mostafaNazari702 force-pushed the esm-migration-phase-4 branch from c8f32f6 to 14d0459 Compare May 8, 2026 23:38
@mostafaNazari702 mostafaNazari702 marked this pull request as ready for review May 9, 2026 01:52
@mostafaNazari702 mostafaNazari702 changed the title chore: migrate to ESM chore: partial migration to ESM May 9, 2026
@mostafaNazari702 mostafaNazari702 force-pushed the esm-migration-phase-4 branch 2 times, most recently from f4d781c to 07e7073 Compare May 13, 2026 20:09
Copy link
Copy Markdown
Member

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

Great progress 🙂. As we mentioned in #5400 (comment), let's remove any major refactors / test shakeups to just leave the clean straightforward changes. LMK when you feel it's ready for re-review and I'll happily take a look. thanks!

@JoshuaKGoldberg JoshuaKGoldberg added the status: waiting for author waiting on response from OP or other posters - more information needed label May 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: waiting for author waiting on response from OP or other posters - more information needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants