feat: prevent script change trigger website build#42
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR prevents script changes from triggering website builds by adding path filters to GitHub workflows and removing the deploy.sh script. The change ensures that modifications to scripts and documentation don't unnecessarily trigger CI/CD processes.
Key changes:
- Added path filters to ignore scripts directory and README.md in GitHub workflows
- Removed the deploy.sh script entirely from the repository
- Updated .dockerignore to exclude the entire file
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| .github/workflows/push-artifacts.yaml | Added paths-ignore filter to skip builds when scripts/** or README.md change |
| .github/workflows/build-check.yaml | Added paths-ignore filter to skip PR checks when scripts/** change |
| deploy.sh | Completely removed the deployment script |
| .dockerignore | Entire file removed |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| branches: | ||
| - main | ||
| paths-ignore: | ||
| - scripts/** |
There was a problem hiding this comment.
The paths-ignore filter excludes 'scripts/**' but the deploy.sh script being removed was in the root directory, not in a scripts folder. Consider whether this filter correctly matches your intended directory structure.
| - scripts/** | |
| - scripts/** | |
| - *.sh |
| branches: [main] | ||
| types: [opened, synchronize] | ||
| paths-ignore: | ||
| - scripts/** |
There was a problem hiding this comment.
Similar to the push-artifacts workflow, this excludes 'scripts/**' but the removed deploy.sh was in the root directory. Verify this path pattern matches your actual scripts location.
| - scripts/** | |
| - scripts/** | |
| - '*.sh' |
No description provided.