improve naming of new adapter api#13253
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
|
ArmandPhilippot
left a comment
There was a problem hiding this comment.
This is a quick review because there is something I find confusing (see my long comment...) and I think I need to better understand that before doing a more thorough review! 😄 But, overall, this looks good!
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
ArmandPhilippot
left a comment
There was a problem hiding this comment.
Thanks again for the explanation, this is much clearer now! I left a few more suggestions, then this seems in a good shape to me.
I know the deprecation is not yet acted, but if this is deprecated... are we sure this will be removed in Astro 7? I mean this could be deprecated but we decide to wait longer (Astro 8) because some edge cases may arise. (I'm asking this because of all the "Astro 7.0 preview")
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
|
Matthew approved the code PR so the deprecation is validated |
ArmandPhilippot
left a comment
There was a problem hiding this comment.
Looks good to me, I only left suggestions about the asides (cautions instead of tips, and maybe they can be reworded now that args and exports are also deprecated).
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
| }); | ||
| } | ||
| ``` | ||
| **Type:** `() => App` |
There was a problem hiding this comment.
And just a reminder that if/when withastro/astro#15483 is approved, we would need to update that too!
There was a problem hiding this comment.
I think ideally it would be better to merge this PR (and associated code PR) first, then make another docs PR for streaming
sarah11918
left a comment
There was a problem hiding this comment.
Nice work Florian (and Armand)! I didn't preview e.g. the updates to the line highlighting in favour of simply tackling the text. See what you think of my comments!
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
|
Bump! |
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
|
OK, as long as my last change looks good in the deploy preview (visually), this will be fine! Moving on to changesets! |
Description (required)
Related issues & labels (optional)
For Astro version:
6.x. See astro PR #15461.