Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

jycouet
Copy link
Contributor

@jycouet jycouet commented Aug 31, 2025

Closes: #651

Copy link

changeset-bot bot commented Aug 31, 2025

🦋 Changeset detected

Latest commit: a0c58fc

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

Copy link

pkg-pr-new bot commented Aug 31, 2025

Open in StackBlitz

npx https://pkg.pr.new/sveltejs/cli/sv@682
npx https://pkg.pr.new/sveltejs/cli/svelte-migrate@682

commit: a0c58fc

Copy link
Member

@manuel3108 manuel3108 left a comment

Choose a reason for hiding this comment

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

Any reason for not moving @node-rs/argon2 to dev in lucia? Seems to be working for me locally.

Otherwise, this looks perfect.

@jycouet
Copy link
Contributor Author

jycouet commented Sep 5, 2025

Any reason for not moving @node-rs/argon2 to dev in lucia? Seems to be working for me locally.

Otherwise, this looks perfect.

I think it's working in development (pnpm dev), but you should not be able to build (pnpm build) if it's in devDependencies therefore it has to stay in dependencies

@manuel3108
Copy link
Member

build and preview work for me in this case

@jycouet
Copy link
Contributor Author

jycouet commented Sep 5, 2025

Ha, found it!

You need @node-rs/argon2 in dependencies if you have @sveltejs/adapter-node. If not, the build will fail.

Copy link
Member

@manuel3108 manuel3108 left a comment

Choose a reason for hiding this comment

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

Can confirm!
Thank you!

@manuel3108 manuel3108 merged commit 2287ac1 into main Sep 6, 2025
8 checks passed
@manuel3108 manuel3108 deleted the chore/depsTodevDeps branch September 6, 2025 12:11
@github-actions github-actions bot mentioned this pull request Sep 5, 2025
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.

Review dependencies vs devDependencies
2 participants