Automattic / Automattic/jetpack
Changelogger Enhancement: Add confirmation messages for successful commands
- Dominant language
- PHP
- Stars
- 1.8k
- Forks
- 898
- Avg merge
- 1d 18h
- Merged PRs (30d)
- 774
Description
### Impacted plugin
None / Other
### What
Add confirmation message when a command is successful
### How
When working on implementing Jetpack Changelogger for WCPay https://github.com/Automattic/woocommerce-payments/pull/3685#discussion_r804523415, we have a feedback that it would be less confusing if we add confirmation messages for successful commands.
Currently, running `./vendor/bin/changelogger write` or `./vendor/bin/changelogger add` simply finishes itself without any confirmation.
For example, I checked the `add` command code [src1](https://github.com/Automattic/jetpack/blob/master/projects/packages/changelogger/src/AddCommand.php#L76-L90) and [src2](https://github.com/Automattic/jetpack/blob/c2f947b546923a4e428e1d886936d7decdf46f32/projects/packages/changelogger/src/AddCommand.php#L335) there is no option to show the successful messages.
Contributor guide
Research direction
Start by reading projects/packages/changelogger/src/AddCommand.php around the referenced command implementation, then run ./vendor/bin/changelogger add and ./vendor/bin/changelogger write to observe their current output. Done means both successful commands clearly report confirmation messages without changing their existing behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- cli
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100