Skip to content

chore: replace tar-fs with modern-tar - #1209

Open
sacrosanctic wants to merge 4 commits into
sveltejs:mainfrom
sacrosanctic:modern-tar
Open

chore: replace tar-fs with modern-tar#1209
sacrosanctic wants to merge 4 commits into
sveltejs:mainfrom
sacrosanctic:modern-tar

Conversation

@sacrosanctic

Copy link
Copy Markdown
Contributor

Closes #

Description

should be a dropin replacement

Checklist

  • Update snapshots (if applicable)
  • Add a changeset (if applicable)
  • Allow maintainers to edit this PR
  • I care about what I'm doing, no matter the tool I use (Notepad, Sublime, VSCode, AI...)

@pkg-svelte-dev

pkg-svelte-dev Bot commented Aug 1, 2026

Copy link
Copy Markdown

Install the latest version of sv from f66b830:

pnpm add https://pkg.svelte.dev/sv/c/f66b830c76c3af7474484c870e79d7f2d39e732c

Open in pkg.svelte.dev: https://pkg.svelte.dev/repos/cli/pr/1209

Note

This PR is from a fork. A maintainer must approve approve each commit before it can be built and installed.

@changeset-bot

changeset-bot Bot commented Aug 1, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: f66b830

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

This PR includes changesets to release 1 package
Name Type
sv 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

Comment thread tsdown.config.ts Outdated
Comment thread packages/sv/src/core/fetch-packages.ts Outdated

@AdrianGonz97 AdrianGonz97 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We're missing a changeset for this, but aside from that, everything looks good!

@Conduitry

Copy link
Copy Markdown
Member

I don't think we really need a changeset, unless we're worried it's not actually a drop-in replacement and mentioning this in the changelog will make people waste less time scratching their heads over why stuff is breaking.

@AdrianGonz97

Copy link
Copy Markdown
Member

hmmm, unless I'm blind, their README doesn't explicitly say that it's a drop-in replacement for tar-fs. In that case, I think adding a changeset should be fine just in case issues do crop up.

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.

4 participants