ci(build): added build test for PRs, removed PRs from push WF#102
ci(build): added build test for PRs, removed PRs from push WF#102ulnanlu wants to merge 13 commits intothezak48:developfrom
Conversation
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
There was a problem hiding this comment.
Pull Request Overview
This PR separates Docker build workflows by creating a dedicated test workflow for pull requests while simplifying the main Docker build workflow for pushes and tags. This allows for faster PR validation without publishing images.
- Created a new test-only workflow for PR validation with smoke testing
- Removed PR triggers from the main Docker build workflow
- Simplified path ignore patterns using wildcard notation
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
.github/workflows/docker-build.yml |
Removed PR triggers and simplified path ignore patterns to only handle pushes/tags |
.github/workflows/docker-build-test.yml |
New workflow for PR testing with Docker build validation and smoke tests |
|
i think i'm done with everything. |
|
the title. no idea if it works...