CI/CD Pipeline, Docker Support & Integration Tests#7
Merged
Conversation
- Add lein-cloverage for code coverage reporting - Add coverage failure threshold at 70% - Add lein-zprint for code formatting checks - Configure codecov integration
- Add test.yml: Run tests on Java 17 and 21 - Add quality.yml: Code linting and coverage reporting - Add release.yml: Automated release deployment - Cache Maven dependencies for faster builds - Upload test results as artifacts
- Add Dockerfile with Alpine Linux base for small image size - Add docker-compose.yml with Zookeeper, Kafka, Datomic services - Add .dockerignore to reduce build context - Configure health checks for all services - Set up proper environment variables and networking
- Add CHANGELOG.md with version history - Add CONTRIBUTING.md with developer guidelines - Include contribution workflow and best practices - Add code style guidelines and testing requirements - Document architecture and git workflow
- Add bug_report.yml template - Configure required fields for issue submission - Include environment and reproduction information
- Fix create-routes call to use repository and publisher separately - Remove unused create-context function - Use routes directly with jetty/run-jetty - Remove wrap-application middleware call from core initialization
- Change create-routes signature from [context] to [repository publisher] - Create context internally from separate adapters - Simplify route composition with proper function signatures
- Change list-all from named arguments to positional [limit offset] - Match http_handler call pattern - Simplify pagination handling
- Remove unused Future import - Fix callback binding to use _ for unused parameter - Move docstring to correct position before parameters - Add return values to try blocks - Fix try-catch syntax in all publisher operations
- Add integration tests for complete notification workflows - Test create, retrieve, list, error handling, and health check - Use mock repository and publisher adapters - Verify HTTP status codes and response formats - Test data persistence and retrieval through full stack
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
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.
Description
This PR implements GitHub Actions workflows for automated testing, Docker containerization, and integration tests for EventDispatch. Includes production-ready configuration and documentation.
Changes
CI/CD & GitHub Actions
Docker Support
Documentation
Code Fixes
Integration Tests
Testing
All tests passing: