-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
feat(astro): experimental astro:env #10974
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Bjorn Lu <bjornlu.dev@gmail.com> Co-authored-by: Emanuele Stoppa <my.burning@gmail.com>
Co-authored-by: Bjorn Lu <bjornlu.dev@gmail.com> Co-authored-by: Emanuele Stoppa <my.burning@gmail.com>
Co-authored-by: Bjorn Lu <bjornlu.dev@gmail.com> Co-authored-by: Emanuele Stoppa <my.burning@gmail.com>
🦋 Changeset detectedLatest commit: 593c254 The changes in this PR will be included in the next version bump. 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 |
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.
This PR is blocked because it contains a minor
changeset. A reviewer will merge this at the next release if approved.
!preview astro-env |
Snapshots have been released for the following packages:
Publish Log
Build Log
|
!preview astro-env |
Snapshots have been released for the following packages:
Publish Log
Build Log
|
!preview astro-env |
Snapshots have been released for the following packages:
Publish Log
Build Log
|
!preview astro-env |
Snapshots have been released for the following packages:
Publish Log
Build Log
|
!preview astro-env |
Snapshots have been released for the following packages:
Publish Log
Build Log
|
!preview astro-env |
Snapshots have been released for the following packages:
Publish Log
Build Log
|
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.
This looks great @florian-lefebvre! I've made a few rewording suggestions. Feel free to usr or modify as needed! 😁
Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>
Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>
Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>
Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>
!preview astro-env |
Snapshots have been released for the following packages:
Publish Log
Build Log
|
Changes
astro:env
. View RFC Astro Env roadmap#894What to review
I'm mostly looking for reviews on
astro:env/setup
: https://github.com/withastro/astro/pull/10974/files#diff-d08d0989a863f87bfceda052de858b8d03124b6bfbff8e084ffbef31240bb446, https://github.com/withastro/astro/pull/10974/files#diff-a95d41705ded14985c748b640b26f3226a7c37d714a8f2fe6171252a59e9941eR79-R81, https://github.com/withastro/astro/pull/10974/files#diff-f982c118ef9afa9c2d69b474b2e206e460b62068cccb0061a5b4eccfe81c708bAlso worth reviewing the adapters pr to see how it will be used for real.
Testing
Unit and integration tests
Docs
Added changeset and config reference