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

docs: more build docs #8500

Closed
wants to merge 2 commits into from
Closed

docs: more build docs #8500

wants to merge 2 commits into from

Conversation

dummdidumm
Copy link
Member

@dummdidumm dummdidumm commented Jan 13, 2023

Transforming the adapters document into a more general build doc
Closes #8436

The alternative to this is to introduce another top level document (keeping the adapter doc as is) - I'm fine with that, too.
Maybe I actually prefer that; opened #8501 for comparison

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

  • It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
  • This message body should clearly illustrate what problems it solves.
  • Ideally, include a test that fails without this PR but passes with it.

Tests

  • Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

  • If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

@changeset-bot
Copy link

changeset-bot bot commented Jan 13, 2023

🦋 Changeset detected

Latest commit: b588918

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

This PR includes changesets to release 6 packages
Name Type
@sveltejs/adapter-auto Patch
@sveltejs/adapter-cloudflare Patch
@sveltejs/adapter-node Patch
@sveltejs/adapter-static Patch
create-svelte Patch
@sveltejs/kit 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

dummdidumm added a commit that referenced this pull request Jan 13, 2023
Transforming the adapters document into a more general build doc
closes #8500
dummdidumm added a commit that referenced this pull request Jan 13, 2023
dummdidumm added a commit that referenced this pull request Jan 13, 2023
@dummdidumm dummdidumm mentioned this pull request Jan 13, 2023
5 tasks
Transforming the adapters document into a more general build doc
closes #8500
@benmccann
Copy link
Member

I really hate that we have to bump the adapter packages everytime we update their docs. Maybe we should take this opportunity to move their docs from the READMEs to the docs folder?

That gives me another idea which would be to create a Building Your App section. Within that could live the new page, the general adapters docs, and docs for each individual official adapter.

@Rich-Harris
Copy link
Member

closing in favour of #8531

Rich-Harris added a commit that referenced this pull request Jan 18, 2023
* docs: more build docs

Transforming the adapters document into a more general build doc
closes #8500

* fix

* docs: move adapter docs to site

* update page titles

* ignore typescript errors

* revert url change

* some more reversions

* reorder sidebar

* tweaks

* rename to build and deploy

* fix casing

* tweaks

* separate out writing section

* tweaks

* formatting

* tweaks

* expose App.Platform automatically if adapter-cloudflare-workers is installed

* Update .changeset/hungry-singers-tease.md

* Update .changeset/hungry-singers-tease.md

* fix links

* amend App.Platform

* fix, hopefully

* tweaks

Co-authored-by: Simon Holthausen <simon.holthausen@vercel.com>
Co-authored-by: Rich Harris <hello@rich-harris.dev>
Co-authored-by: Rich Harris <richard.a.harris@gmail.com>
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.

Prerender triggered, even when specified to disable
3 participants