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

Adding docs for the new astro:build:generated hook #1572

Merged
merged 6 commits into from
Sep 22, 2022

Conversation

tony-sull
Copy link
Contributor

@tony-sull tony-sull commented Sep 15, 2022

What kind of changes does this PR include?

  • Minor content fixes (broken links, typos, etc.)
  • New or updated content

Description

  • Adding docs for the new hook from PR #4775
  • Updates the astro:build:ssr hook docs, it is never fired for SSG builds

@netlify
Copy link

netlify bot commented Sep 15, 2022

Deploy Preview for astro-docs-2 ready!

Name Link
🔨 Latest commit d9bf787
🔍 Latest deploy log https://app.netlify.com/sites/astro-docs-2/deploys/632c7dbc1ae7f10008b763d4
😎 Deploy Preview https://deploy-preview-1572--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@yanthomasdev
Copy link
Member

LGTM!

@sarah11918 sarah11918 added the add new content Document something that is not in docs. May require testing, confirmation, or affect other pages. label Sep 21, 2022
Copy link
Member

@delucis delucis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems solid @tony-sull!

A suggestion and a query for you nonetheless 🙌

Tony Sullivan and others added 2 commits September 21, 2022 14:42
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Chris Swithinbank <swithinbank@gmail.com>
@sarah11918
Copy link
Member

Note to all of us that astro PR #4775 has been "approved" but not merged. This shouldn't merge before the hook is active, but also, SHOULD merge WHEN the hook is active. Everyone keep their eyes open! 😄

@sarah11918 sarah11918 added the merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) label Sep 21, 2022
@tony-sull
Copy link
Contributor Author

Updated note: PR #4775 was just merged in and will go out in tomorrow's minor release 🎉 We'll be ready to merge this in once the release is cut

Copy link
Member

@delucis delucis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@sarah11918 sarah11918 merged commit 5197abc into main Sep 22, 2022
@sarah11918 sarah11918 deleted the feat/generated-hook branch September 22, 2022 15:27
nokazn pushed a commit to nokazn/docs.astro.build that referenced this pull request Oct 2, 2022
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Chris Swithinbank <swithinbank@gmail.com>
@sarah11918 sarah11918 added the minor-release For the next minor release; in the milestone, "merge queue" when approved by Sarah! label Jun 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add new content Document something that is not in docs. May require testing, confirmation, or affect other pages. merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) minor-release For the next minor release; in the milestone, "merge queue" when approved by Sarah!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants