Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Sep 2, 2025

  • Create CI workflow for automated checks on pull requests
  • Add security audit step with npm audit
  • Add TypeScript and Astro validation step
  • Add build verification step
  • Use same Bun setup as deployment workflow
  • Replace npm/npx commands with bun equivalents per feedback
    • Changed npm audit --audit-level=high to bun audit --audit-level=high
    • Changed npx astro check to bunx astro check

💬 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: t128n <53407297+t128n@users.noreply.github.com>
@t128n
Copy link
Owner

t128n commented Sep 2, 2025

@copilot Make sure to use bun instead of node.js

Co-authored-by: t128n <53407297+t128n@users.noreply.github.com>
Copilot AI changed the title [WIP] Like the deploy.yml, add a ci.yml which runs automated checks for this project when someone opens up a pr against main. Add CI workflow for automated checks on pull requests Sep 2, 2025
Copilot AI requested a review from t128n September 2, 2025 10:49
Co-authored-by: t128n <53407297+t128n@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Sep 2, 2025

@copilot Make sure to use bun instead of node.js

Updated the CI workflow to use bun commands instead of node.js:

  • Changed npm audit --audit-level=high to bun audit --audit-level=high
  • Changed npx astro check to bunx astro check

These changes ensure the CI workflow uses bun consistently throughout. (00e4d2e)

@t128n t128n marked this pull request as ready for review September 2, 2025 11:04
@t128n t128n merged commit 0b06e70 into main Sep 2, 2025
1 check passed
@t128n t128n deleted the copilot/fix-6c0ac3af-24ac-49d3-84bf-8e039fc9a2f1 branch October 23, 2025 16:31
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