-
Notifications
You must be signed in to change notification settings - Fork 2
Fix/deployment integration tests #109
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
Conversation
create-app-working use getEnv for build_sha update astro packages
prod invocation integration test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR fixes deployment integration tests by upgrading Astro documentation dependencies and adding a new production invocation workflow. The changes upgrade Astro from v4 to v5 and Starlight from v0.26 to v0.37, introduce a new test application for live environment testing, and implement a GitHub Actions workflow to build, deploy, and verify the test application in production/preprod environments.
Changes:
- Upgraded Astro (v4.16.19 → v5.16.9) and Starlight (v0.26.1 → v0.37.2) with corresponding configuration updates
- Added new integration test application and deployment workflow for live environment validation
- Updated ESLint and gitignore configurations to accommodate new test files
Reviewed changes
Copilot reviewed 10 out of 13 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| pnpm-workspace.yaml | Removed unnecessary quotes from package path |
| pnpm-lock.yaml | Updated lockfile with new Astro v5, Starlight v0.37, and related dependencies |
| integration-tests/test-application/test-app.js | New test application for production environment validation |
| integration-tests/test-application/.gitkeep | Placeholder for new test directory |
| docs/package.json | Updated Astro and Starlight dependency versions |
| docs/astro.config.mjs | Updated social config from object to array format for Starlight v0.37 |
| config/eslint/repo/.eslintrc.cjs | Extended ignore pattern to cover all integration-tests |
| .gitignore | Added test-app.wasm to ignored files |
| .github/workflows/prod-invocation.yaml | New workflow for deploying and testing app in production |
| .github/workflows/deploy.yaml | Integrated prod invocation test into deployment pipeline |
| .github/workflows/build-libs.yaml | Added execute permissions step for bin files |
| .github/scripts/prod-invocation/invoke-test-app.js | Script to invoke and validate deployed test application |
| .github/scripts/prod-invocation/create-test-app.js | Script to build test application WASM binary |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
No description provided.