Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

I propose to pin @sveltejs/adapter-auto and @sveltejs/kit version in create-kit-docs script #50

Closed
stephane-klein opened this issue Jul 10, 2022 · 5 comments

Comments

@stephane-klein
Copy link
Contributor

Hi,

@mihar-22 What do you think about pin @sveltejs/adapter-auto and @sveltejs/kit version here to avoid some version errors like #48 ?

Best regards,
Stéphane

@mihar-22
Copy link
Contributor

We'll just need to update KitDocs to match the latest SvelteKit. Things are changing quickly as they're working towards 1.0 and I'm a little busy to keep up. If anyone opens a PR that does that I'll happily pull it in @stephane-klein :)

@stephane-klein
Copy link
Contributor Author

@mihar-22 thanks ! 👍

Question: do you think it's a bad idea to pin versions? I think that it would save kit-docs users (like me) a lot of trouble in case of SvelteKit breaking change.

@mihar-22
Copy link
Contributor

mihar-22 commented Jul 13, 2022

No problem!

I think you can cause trouble by both pinning and not. I think we have to just look at the likelihood of breaking changes occurring. I don't really see anything else breaking KitDocs because we're working with the very core of SvelteKit (mainly routing and endpoints). The recent CLI and Vite config change was a good thing for the ecosystem and is an example of a core change that won't really happen often.

If we find that it's breaking often we can definitely pin it until 1.0 :)

@stephane-klein
Copy link
Contributor Author

If we find that it's breaking often we can definitely pin it until 1.0 :)

@mihar-22 ok, I understand, we will maybe pin it the next time 🙂

@stephane-klein
Copy link
Contributor Author

@mihar-22 There are another breaking change #52, I think that vite version must be pinned.

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

No branches or pull requests

2 participants