feat: add Docker support for Quilt MCP #191
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Version bump from 0.6.12 to 0.6.13 - Added CHANGELOG entry for Docker container support features - Documented HTTP transport implementation and tooling
- Extract Docker operations to reusable scripts/docker.sh - Move Docker push from push.yml to create-release action - Add manual Docker targets to make.deploy (docker-build, docker-push) - Update env.example with Docker configuration variables - Optimize CI/CD by skipping Docker for dev releases - Add PR validation for Docker builds without pushing - Document required GitHub secrets for ECR operations
- Merged docker.sh and docker_image.py into single docker.py script - Unified tag generation, build, and push operations - Updated all references in workflows and Makefiles - Added comprehensive tests for the new unified script - Removed legacy bash and separate Python scripts - Maintains backward compatibility with same functionality
- Move all Docker build/push logic to create-release action - Reset push.yml and pr.yml to cleaner state from main - Production releases build and push Docker via create-release - Dev releases skip Docker (build-docker: false) - PR builds test Docker build without pushing - Fix script reference to use docker.py instead of docker.sh
- Add 'info' command to docker.py for retrieving image URIs - Update create-release action to use docker.py info command - Add comprehensive tests for the new info command - Ensure GitHub Actions output format support
- Add update_release_notes() to append status to GitHub releases - Implement find_pr_for_sha() for PR detection - Prioritize release notes updates (critical) over PR comments (graceful) - Add comprehensive test coverage following TDD principles - Update create-release action to output release ID - Integrate with GitHub Actions workflows
- All parameters now passed explicitly from GitHub Action - No environment variable dependencies in script - is-production flag determines release type (not tag pattern) - package-name parameter for flexibility - Improved testability with pure function approach - Updated tests to match new design - Decision logic moved to GitHub Action
- Add --is-production and --package-name arguments to script - Fix test mocking approach for sys.argv - Core functionality tests all passing
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Testing
Closes #188