-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
docs: update editor setup section #15010
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
base: main
Are you sure you want to change the base?
Conversation
|
|
The Editor Support page now lives in this collection: https://sveltesociety.dev/collection/editor-support-c85c080efc292a34 |
|
Hello sir,thanks for pointing it out! Please let me know if you'd like any further tweaks. |
|
Thanks for catching this @dev-singh12. Would you also be able to send a PR to the svelte repo? @kevmodrome there's also broken links to the svelte society site on https://svelte.dev/docs/kit/additional-resources, https://svelte.dev/docs/svelte/faq, and https://svelte.dev/docs/svelte/getting-started#Editor-tooling. Can you share what the links should be updated to there? |
|
Sure @benmccann sir, happy to help with that! Once @kevmodrome sir confirms the correct replacement URLs for the broken Svelte Society links, I can open a PR on the svelte repo to update them as well. Just share the final links and I’ll prepare the follow-up PR right away. |
|
@benmccann sir,I have a raised a pr to the svelte repo (sveltejs/svelte#17290) |
|
Ok. I've finally managed to find some time to look at the links. Here's the templates one: https://sveltesociety.dev/recipe/sveltekit-templates-and-examples-e789ed397e7f38fc |
Fixes #14941
This PR updates the “Editor setup” section in the SvelteKit documentation to remove an outdated Svelte Society link that no longer provides editor support information. The previous text implied broad editor support via that page, but the page has been empty for some time.
What this PR fixes
This avoids confusing users and reflects the current state of editor tooling more accurately.
This is a documentation-only change; no package version bump is required, so no changeset is needed.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
Tests
pnpm testand lint the project withpnpm lintandpnpm checkChangesets
pnpm changesetand following the prompts.Edits