-
Notifications
You must be signed in to change notification settings - Fork 3
Refactor workflows into semantic jobs and add workflow orchestration #24
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
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: zimme <[email protected]>
|
@copilot add workflowndispatch to build workflow. Split making change and commiting change into separate jobs in update workflow it that makes semantic sense. After commit, update workflow should trigger build workflow. |
- Add workflow_dispatch trigger to docker-build-and-publish workflow - Split update workflow into separate update-dockerfile and commit-changes jobs - Add workflow trigger from update to build workflow after commit - Improve job naming and semantic separation Co-authored-by: zimme <[email protected]>
All requested changes have been implemented in commit 68621ad:
The workflows now have better semantic separation and the update workflow will automatically trigger the build workflow after successfully committing changes. |
|
@copilot the workflow dispatch can probably also be a separate job? |
Co-authored-by: zimme <[email protected]>
Good suggestion! I've separated the workflow dispatch trigger into its own dedicated
The |
This PR refactors both GitHub Actions workflows to improve maintainability, debugging, and workflow orchestration through better semantic job separation.
Changes Made
docker-build-and-publish.yaml Refactoring
Before: Single
dockerJobThe original workflow had one large job with multiple unrelated steps mixed together:
After: Three Focused Jobs
1.
prepareJob2.
build-testJob--versioncheck)preparejob3.
publishJobprepareandbuild-testjobscheck-for-updates-and-update-dockerfile.yaml Refactoring
Before: Two Jobs with Mixed Responsibilities
The original workflow had version checking and file updates/commits mixed together.
After: Four Semantic Jobs
1.
check-version-updatesJob2.
update-dockerfileJob3.
commit-changesJob4.
trigger-buildJobWorkflow Orchestration Improvements
workflow_dispatchto build workflow for manual executionBenefits
Technical Details
Fixes #23.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.