Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: upgrade mrmime and sirv to modernize javascript mime type #11419

Merged
merged 1 commit into from
Dec 20, 2023

Conversation

benmccann
Copy link
Member

closes #11393

Copy link

changeset-bot bot commented Dec 20, 2023

🦋 Changeset detected

Latest commit: 5cef28c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@sveltejs/adapter-node Patch
@sveltejs/kit Patch

Not sure what this means? Click here to learn what changesets are.

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

@Conduitry
Copy link
Member

There are a bunch of lockfile changes here that don't look right. Why are there two versions of esbuild now?

@Conduitry
Copy link
Member

I've gone ahead an regenerated the lockfile and pushed 548340c. This looks good to me now. We still have mrmime@1 in the tree, but it looks like that's unavoidable until worktop upgrades.

@benmccann
Copy link
Member Author

I just pushed the same change because I saw your first comment and not your second yet. I don't know why pnpm does screwy things when you run pnpm update...

@benmccann
Copy link
Member Author

This looks good to me now.

Ok, I'll go ahead and merge then

@benmccann benmccann merged commit 2a091dc into main Dec 20, 2023
@benmccann benmccann deleted the mime-sirv branch December 20, 2023 17:31
@github-actions github-actions bot mentioned this pull request Dec 20, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

adapter-node return "application/javascript" for mjs instead "text/javascript"
2 participants