Skip to content

Issue #561: add astro check#562

Merged
brian-montgomery merged 2 commits into
mainfrom
lsr-explore/gh-561/add-astro-check
May 21, 2026
Merged

Issue #561: add astro check#562
brian-montgomery merged 2 commits into
mainfrom
lsr-explore/gh-561/add-astro-check

Conversation

@lsr-explore
Copy link
Copy Markdown
Contributor

@lsr-explore lsr-explore commented May 10, 2026

Fixes

Description

  • Add astro check to dev dependencies
  • Add script to run astro check
  • Add to github pages

Note

The job will look green because I put the continue-on-error on the astro:check run so the deploy will not be blocked if main is configured to not deploy if a job fails.

Screen capture

screenshot of checks showing that the astro check ran

Astro check run

See astro-check job for errors and warnings
screenshot of the details of the astro check

@netlify
Copy link
Copy Markdown

netlify Bot commented May 10, 2026

Deploy Preview for accessiblecommunity canceled.

Name Link
🔨 Latest commit 1f4c02c
🔍 Latest deploy log https://app.netlify.com/projects/accessiblecommunity/deploys/6a0f486979d23400085c6a1d

@lsr-explore lsr-explore force-pushed the lsr-explore/gh-561/add-astro-check branch 2 times, most recently from c3b7d85 to d032168 Compare May 10, 2026 18:41
@lsr-explore lsr-explore force-pushed the lsr-explore/gh-561/add-astro-check branch from d032168 to b50e122 Compare May 10, 2026 18:44
@lsr-explore lsr-explore linked an issue May 10, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

@brian-montgomery brian-montgomery left a comment

Choose a reason for hiding this comment

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

Laurie, thank you for this. It finds a ton of errors.

I updated the Makefile to be able to run this and added quick permissions check for safety.

@brian-montgomery brian-montgomery merged commit f7465e0 into main May 21, 2026
10 checks passed
@brian-montgomery brian-montgomery deleted the lsr-explore/gh-561/add-astro-check branch May 21, 2026 18:06
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.

Add astro check support

2 participants