Skip to content

fix: ensure filename for esm format - #364

Merged
egoist merged 2 commits into
masterfrom
egoist-patch-1
Jun 21, 2020
Merged

fix: ensure filename for esm format#364
egoist merged 2 commits into
masterfrom
egoist-patch-1

Conversation

@egoist

@egoist egoist commented Jun 21, 2020

Copy link
Copy Markdown
Owner

fix regression for #363, closes #363

@vercel

vercel Bot commented Jun 21, 2020

Copy link
Copy Markdown

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/egoist/bili/75a3iilfn
✅ Preview: https://bili-git-egoist-patch-1.egoist.vercel.app

@codecov

codecov Bot commented Jun 21, 2020

Copy link
Copy Markdown

Codecov Report

Merging #364 into master will decrease coverage by 0.11%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #364      +/-   ##
==========================================
- Coverage   75.33%   75.21%   -0.12%     
==========================================
  Files          11       11              
  Lines         454      456       +2     
  Branches      168      169       +1     
==========================================
+ Hits          342      343       +1     
- Misses        107      108       +1     
  Partials        5        5              
Impacted Files Coverage Δ
src/index.ts 73.68% <66.66%> (-0.16%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4aae8f0...2c23351. Read the comment docs.

Comment thread src/index.ts Outdated
.replace(/\[ext\]/, '.js')

if (rollupFormat === 'esm') {
fileName = fileName.replace(/\[format\]/, '.esm')

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
fileName = fileName.replace(/\[format\]/, '.esm')
fileName = fileName.replace(/\[format\]/, 'esm')

Comment thread src/index.ts
@egoist
egoist merged commit fe897b2 into master Jun 21, 2020
@egoist
egoist deleted the egoist-patch-1 branch June 21, 2020 11:42
@egoist

egoist commented Jun 21, 2020

Copy link
Copy Markdown
Owner Author

🎉 This PR is included in version 5.0.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

chocolateboy added a commit to chocolateboy/conjoin that referenced this pull request Jun 23, 2020
chocolateboy added a commit to chocolateboy/conjoin that referenced this pull request Jun 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Breaking change of ESM format filenames

2 participants