Run check on all addons test
#569This should prevent stuff like #341 to happen again
Latest commit: a1d8bdb
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
npx https://pkg.pr.new/sveltejs/cli/sv@569
npx https://pkg.pr.new/sveltejs/cli/svelte-migrate@569
commit: a1d8bdb
@manuel3108 this is the prototype for making check pass in all addons
Edit: Wont need the tooling comment change
Edit: The paraglide and Lucia hook are different. Would need to update paraglide to magic string implementation to match them. Running parseScript removes jsdoc comments from previous run. In this case paraglide comments were getting overridden when Lucia hook.server ran with ast.
Morning @manuel3108 I haven't added the check, the draft was purely to get the maintainers eyes on it make sure I'm sticking to the vision of the code base here since I'm fundamentally changing the Lucia adds hooks implementation from ast to strings. If I get the nod and go ahead once we work out the creases I'll go through and move paraglide to string style and implement running 'pnpm check' in all add-ons test.
Note: there hasn't been movement on storybook fix upstream yet.