Skip to content

v2.1.0 — AWS API alignment, Docker dev env, contributor PRs#130

Merged
maxbanton merged 13 commits into
masterfrom
feature/v2.1-fixes
May 1, 2026
Merged

v2.1.0 — AWS API alignment, Docker dev env, contributor PRs#130
maxbanton merged 13 commits into
masterfrom
feature/v2.1-fixes

Conversation

@maxbanton

@maxbanton maxbanton commented Apr 26, 2026

Copy link
Copy Markdown
Owner

v2.1.0 maintenance release. No breaking changes.

Closes #95, #101, #103, #111, #114. Adopts #102, #115.

@maxbanton maxbanton force-pushed the feature/v2.1-fixes branch 2 times, most recently from 7d4be08 to a97787f Compare April 26, 2026 20:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Maintenance release to align the CloudWatch handler with post-2022 AWS CloudWatch Logs API behavior (sequence token deprecation, init flow changes, larger event splitting), while adding a Docker-based dev/CI environment and updating docs/changelog accordingly.

Changes:

  • Update CloudWatch handler: remove sequence token + self-throttle, raise event chunk limit, switch init to idempotent create* with ResourceAlreadyExistsException handling, and add flush()/reset() flushing.
  • Rewrite unit tests to match the new initialization flow and new flush/reset behavior.
  • Add Docker/Compose/Makefile-based dev+CI setup and update documentation/changelog to reflect the release.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
src/Handler/CloudWatch.php AWS API alignment (no sequence token, no self-throttle), idempotent init, new flush() + reset() behavior.
tests/Handler/CloudWatchTest.php Updates tests to cover new init strategy and flush/reset behavior.
README.md Adds IAM Task Role guidance and updates IAM permissions section to remove reliance on Describe*.
CHANGELOG.md Introduces changelog entries for v2.1.0 release contents.
docker/Dockerfile Adds a PHP CLI image for containerized dev/CI (Composer + pcov).
compose.yaml Defines PHP version matrix services for local/dev CI parity.
Makefile Adds standardized Docker-driven commands for build/lint/test/matrix/coverage.
.github/workflows/php.yml Replaces setup-php based CI with container-based CI driven by make.
.gitattributes Excludes dev/CI files from Composer dist exports.
.dockerignore Shrinks Docker build context.
.gitignore Ignores generated build/ artifacts.
composer.json Reformats JSON and updates package metadata.
Comments suppressed due to low confidence (2)

Makefile:35

  • The analyse target calls vendor/bin/phpstan, but composer.json doesn’t currently require phpstan in require-dev. Either add the dependency (e.g., phpstan/phpstan) or adjust/remove this target so make analyse works after make install.
analyse:
	$(DC) vendor/bin/phpstan analyse --level=8 src/

Makefile:26

  • test-coverage writes to build/logs/clover.xml, but the Makefile never creates build/logs. On a clean checkout this typically causes PHPUnit to fail to open the file and the workflow’s upload step will error. Add a step to create the directory before running PHPUnit (or write to an existing path).
test-coverage:
	$(DC) vendor/bin/phpunit --coverage-text --coverage-clover build/logs/clover.xml


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread CHANGELOG.md Outdated
Comment thread src/Handler/CloudWatch.php
Comment thread src/Handler/CloudWatch.php
Comment thread tests/Handler/CloudWatchTest.php
Comment thread src/Handler/CloudWatch.php Outdated
Comment thread README.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread src/Handler/CloudWatch.php
@maxbanton maxbanton force-pushed the feature/v2.1-fixes branch 2 times, most recently from e813321 to 1218f51 Compare April 26, 2026 21:11
@maxbanton maxbanton force-pushed the feature/v2.1-fixes branch from d717dbb to e6feaf6 Compare May 1, 2026 21:33
@maxbanton maxbanton marked this pull request as ready for review May 1, 2026 22:26
@maxbanton maxbanton merged commit ca172cb into master May 1, 2026
5 checks passed
@maxbanton maxbanton deleted the feature/v2.1-fixes branch May 1, 2026 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Logs are not sent in Laravel queues

2 participants