Skip to content

[WIP] Address feedback from review on starlight docs PR#755

Merged
pelikhan merged 4 commits intopelikhan/astrofrom
copilot/stack-pr-748-2
Sep 12, 2025
Merged

[WIP] Address feedback from review on starlight docs PR#755
pelikhan merged 4 commits intopelikhan/astrofrom
copilot/stack-pr-748-2

Conversation

Copy link
Contributor

Copilot AI commented Sep 11, 2025

Migrating docs to starlight.

  • scafold starlight project
  • move markdown
  • add frontmatter
  • updated index.md
  • update readme to be AI friendly
  • apply review comments to docs.yml workflow
  • configure docs workflow to build on push but deploy only on workflow_dispatch and release events

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
path: ./docs/dist

deploy:
if: github.event_name == 'workflow_dispatch' || github.event_name == 'release' || github.event_name == 'push'
Copy link
Contributor

Choose a reason for hiding this comment

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

do not run deploy on push
@copilot

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed push event from deploy job condition. Now the deploy job only runs on workflow_dispatch and release events, while the build job still runs on push to docs/** for validation. (commit: 125eb5f)

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan pelikhan marked this pull request as ready for review September 12, 2025 00:26
@pelikhan pelikhan merged commit 53321d0 into pelikhan/astro Sep 12, 2025
@pelikhan pelikhan deleted the copilot/stack-pr-748-2 branch September 12, 2025 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants