Skip to content

build: update package.json entries with explicit mjs / mts extensions #1386

Merged
michaelfaith merged 2 commits intomainfrom
build/fix-tsdown
Nov 13, 2025
Merged

build: update package.json entries with explicit mjs / mts extensions #1386
michaelfaith merged 2 commits intomainfrom
build/fix-tsdown

Conversation

@michaelfaith
Copy link
Collaborator

@michaelfaith michaelfaith commented Nov 13, 2025

PR Checklist

Overview

(This was failing in the renovate PR: #1373)

Following rolldown/tsdown#517, fixedExtension is now on by default when the platform is node, which means that it's emitting mjs for any esm js files and .d.mts for esm ts files.

renovate bot and others added 2 commits November 12, 2025 18:18
(This was failing in the renovate PR: #1373)

Following rolldown/tsdown#517, fixedExtension is now on by default when the platform is node, which means that it's emitting mjs for any esm js files and .d.mts for esm ts files.
@github-actions
Copy link
Contributor

github-actions bot commented Nov 13, 2025

πŸ‘‹ Hi @michaelfaith, thanks for the pull request! A scan flagged some concerns with it. Could you please take a look?

[pr-linked-issue] This pull request is not linked as closing any issues. This repository keeps to GitHub issues for discussing potential changes. Most or all changes should be marked as approved in an issue before a pull request is sent to resolve them. To resolve this report:

  • If this is a straightforward documentation change that doesn't need an issue, you can ignore this report
  • If there is a backing issue, add a 'fixes #...' link to the pull request body
  • Otherwise, file an issue explaining what you'd like to happen

[pr-task-completion] This PR's body is missing [x] checks on the following tasks from the PR template.

Repositories often provide a set of tasks that pull request authors are expected to complete. Those tasks should be marked as completed with a [x] in the pull request description. Please complete those tasks and mark the checks as [x] completed.

πŸ—ΊοΈ This message was posted automatically by OctoGuide: a bot for GitHub repository best practices.

@codecov
Copy link

codecov bot commented Nov 13, 2025

Codecov Report

βœ… All modified and coverable lines are covered by tests.
βœ… Project coverage is 96.84%. Comparing base (b76d804) to head (f091c63).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1386   +/-   ##
=======================================
  Coverage   96.84%   96.84%           
=======================================
  Files          27       27           
  Lines         538      538           
  Branches      159      159           
=======================================
  Hits          521      521           
  Misses          2        2           
  Partials       15       15           

β˜” 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.

@michaelfaith michaelfaith marked this pull request as ready for review November 13, 2025 00:59
@michaelfaith michaelfaith enabled auto-merge (squash) November 13, 2025 01:00
@michaelfaith michaelfaith merged commit c26616a into main Nov 13, 2025
15 of 17 checks passed
@michaelfaith michaelfaith deleted the build/fix-tsdown branch November 13, 2025 01:00
@github-actions
Copy link
Contributor

πŸŽ‰ This is included in version v0.72.0 πŸŽ‰

The release is available on:

Cheers! πŸ“¦πŸš€

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant