-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Document custom mode support #9641
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
Conversation
✅ Deploy Preview for astro-docs-2 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! @bluwy! I stole a bit from the PR changeset, and I think the environment variable stuff should become its own smaller section AFTER introducing variables in general (maybe we even give it a sub heading? ), so see what you think!
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great @bluwy ! Just had a question about adding more info in the CLI reference page. I'm not sure myself, so I'll leave it to you!
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great @bluwy 🙌
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Description (required)
Add documentation for custom mode support. You can now pass any string to
--modeand themodeinline config.It also adds the
--devOutputflag support forastro buildto output a development-based build. Previously I don't believe there's an easy way, even if you had set--mode development(it may only partially work).This is a feature and shouldn't have any breaking changes, so there's no entry in the migration guide.
Related issues & labels (optional)
For Astro version:
5.0. See astro PR withastro/astro#12150