-
Notifications
You must be signed in to change notification settings - Fork 5
push composer changelog #22
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
Conversation
Pull Request SummaryThis PR makes the following changes: CHANGELOG.md
composer.json
composer.lock
This PR appears to be preparing for a new release with Route Plan functionality while updating package metadata. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Important
Looks good to me! 👍
Reviewed everything up to 8709434 in 47 seconds. Click for details.
- Reviewed
1866
lines of code in3
files - Skipped
0
files when reviewing. - Skipped posting
3
draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. CHANGELOG.md:8
- Draft comment:
The new [1.1.0] section is well formatted and follows the 'Keep a Changelog' guidelines. Consider clarifying the 'get endpoint(s)' description if needed. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
2. composer.json:25
- Draft comment:
The authors list now includes 'Shuai Tseng'. This addition is clear; ensure that this update was intentional. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
3. composer.lock:1
- Draft comment:
The composer.lock file changes appear auto‐generated. No manual modifications are needed, but ensure it remains in sync with composer.json. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
Workflow ID: wflow_DR0r3v4A8ghNDErj
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Important
Looks good to me! 👍
Reviewed 9eac805 in 59 seconds. Click for details.
- Reviewed
13
lines of code in1
files - Skipped
0
files when reviewing. - Skipped posting
1
draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. CHANGELOG.md:47
- Draft comment:
Updated the [Unreleased] link to compare against v1.1.0, ensuring it reflects the latest release tag. This is consistent with semantic versioning practices. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =0%
<= threshold50%
This comment is purely informative and does not provide any actionable feedback or suggestions for improvement. It simply states what was done without offering any critique or guidance.
Workflow ID: wflow_AOFFirKhG6VqReYT
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Important
Add version 1.1.0 to changelog with new Route Plan endpoints and update composer files.
CHANGELOG.md
.create
,get
(by id and query),update
,deleteOne
, andaddTasksToRoutePlan
.composer.json
.composer.lock
to lock dependencies.This description was created by
for 9eac805. You can customize this summary. It will automatically update as commits are pushed.