Skip to content

chore: Update version and changelog - #22

Open
kofimokome wants to merge 1 commit into
devfrom
chore/package-version
Open

chore: Update version and changelog#22
kofimokome wants to merge 1 commit into
devfrom
chore/package-version

Conversation

@kofimokome

@kofimokome kofimokome commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Update package version and changelog in preparation for a new minor release

@kofimokome
kofimokome requested a review from nfebe July 20, 2026 09:47
@sourceant

sourceant Bot commented Jul 20, 2026

Copy link
Copy Markdown

Code Review Summary

The PR aims to release a new version of the package with Laravel 12 support. While the metadata updates are present, there are issues with the versioning strategy and a possible omission of dependency constraint updates.

🚀 Key Improvements

  • Improve changelog clarity by focusing on features rather than implementation details.
  • Adopt PHP best practices by removing the redundant version field in composer.json.

💡 Minor Suggestions

  • Standardize spacing in Markdown list items.

🚨 Critical Issues

  • Version mismatch: The PR description specifies a minor release, but the code implements a patch version bump (1.2.1 instead of 1.3.0).
  • Missing dependency updates: The claim of Laravel 12 support is not backed by visible changes to composer.json requirements.

@sourceant sourceant Bot 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.

Review complete. See the overview comment for a summary.

Comment thread CHANGELOG.md Outdated
@kofimokome
kofimokome force-pushed the chore/package-version branch 2 times, most recently from 9834378 to 98d53e2 Compare July 20, 2026 09:49

@sourceant sourceant Bot 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.

Review complete. See the overview comment for a summary.

Comment thread CHANGELOG.md

## [Unreleased]

## [1.2.1] - 2026-07-20

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The PR description indicates this is a 'minor release', which should increment the second digit (1.3.0). Incrementing the third digit (1.2.1) signifies a patch/bug-fix release. Furthermore, updating dependency support is typically categorized under ### Changed rather than ### Added.

Suggested change
## [1.2.1] - 2026-07-20
## [1.3.0] - 2026-07-20
### Changed

Comment thread composer.json
"type": "library",
"license": "proprietary",
"version": "1.2.0",
"version": "1.2.1",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Aligning with the 'minor release' intent and the addition of Laravel 12 support, the version should be bumped to 1.3.0 instead of 1.2.1.

Suggested change
"version": "1.2.1",
"version": "1.3.0",

@kofimokome
kofimokome force-pushed the chore/package-version branch from 98d53e2 to 0b6cc7b Compare July 20, 2026 09:51

@sourceant sourceant Bot 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.

Review complete. No specific code suggestions were generated. See the overview comment for a summary.

@sourceant sourceant Bot 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.

Review complete. See the overview comment for a summary.

Comment thread CHANGELOG.md
## [1.2.1] - 2026-07-20

### Changed
- Update composer.json to support Laravel 12

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Changelog entries are more effective when they describe the feature or compatibility added (e.g., 'Add support for Laravel 12') rather than the specific file modified. Additionally, ensure that the require section in composer.json is actually updated to include Laravel 12 constraints (e.g., ^12.0), as those changes are not visible in the current diff.

Suggested change
- Update composer.json to support Laravel 12
- Add support for Laravel 12

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.

2 participants