-
-
Couldn't load subscription status.
- Fork 2
Upgrade PR previews #116
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
Closed
Closed
Upgrade PR previews #116
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
13d52c4 to
47e2975
Compare
π Documentation PreviewYour changes are live! View the preview deployment: π View Preview β
|
Remove complex metadata file management and redundant steps. Changes: - Remove separate pages-metadata deployment - Remove github-script step for CNAME cleanup - Include .nojekyll and _config.yml directly in dist folder - Simplify to single PR preview deployment step This streamlines the workflow while maintaining all necessary functionality for both main site and PR preview deployments.
Temporarily allow deployment from topic/gh-pr-preview branch to restore the main site while testing PR previews. This will be reverted before merging to main.
Remove topic branch from main site deployment condition. Main site will only deploy from main branch going forward.
- Rename index.mdoc to index.mdx
- Convert Markdoc syntax ({% %}) to MDX JSX syntax
- Import components from @astrojs/starlight/components
- Maintain all existing functionality and content
Use import.meta.env.BASE_URL to properly construct links so they work in both main site (/) and PR previews (/pr-preview/pr-N/).
Replace HTML button (which doesn't render) with clean markdown: - Use heading with large link for preview - Add table for metadata display - Use blockquote for the note - Update cleanup message to match style
- Use π emoji for preview link (better than π) - Change workflow emoji from π to βοΈ for better visual
|
Just an experiment. Dismissing due to high complexity. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Testing