-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat: experimental static import.meta.env #12105
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 project configuration. |
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
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 clear to me! I only spotted a missing rename.
src/content/docs/en/reference/experimental-flags/static-import-meta-env.mdx
Outdated
Show resolved
Hide resolved
…-meta-env.mdx Co-authored-by: Armand Philippot <[email protected]>
I think this looks good! |
src/content/docs/en/reference/experimental-flags/static-import-meta-env.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/en/reference/experimental-flags/static-import-meta-env.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: Yan <[email protected]>
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.
OK, here's a draft! See what you think @florian-lefebvre !
src/content/docs/en/reference/experimental-flags/static-import-meta-env.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/en/reference/experimental-flags/static-import-meta-env.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/en/reference/experimental-flags/static-import-meta-env.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/en/reference/experimental-flags/static-import-meta-env.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/en/reference/experimental-flags/static-import-meta-env.mdx
Outdated
Show resolved
Hide resolved
<Since v="5.13.0" /> | ||
</p> | ||
|
||
:::tip[Astro 6.0 preview] |
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.
Was trying to think of some "tip" to distinguish "here's a new feature" from "this is a new default" because we have two kinds of experimental flags.
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.
A few more nits from me.
src/content/docs/en/reference/experimental-flags/static-import-meta-env.mdx
Outdated
Show resolved
Hide resolved
src/content/docs/en/reference/experimental-flags/static-import-meta-env.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: Yan <[email protected]>
LGTM! |
src/content/docs/en/reference/experimental-flags/static-import-meta-env.mdx
Outdated
Show resolved
Hide resolved
…-meta-env.mdx Co-authored-by: Sarah Rainsberger <[email protected]>
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! Thanks.
Description (required)
Documents
experimental.staticImportMetaEnv
Related issues & labels (optional)
For Astro version:
5.13
. See astro PR #14173.