MPT-17812: rename mpt-tool to mpt-service-cli#36
Conversation
📝 WalkthroughWalkthroughThe pull request renames the CLI tool from Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
07ac92f to
ffed8fa
Compare
ffed8fa to
f47aea5
Compare
|
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
docs/PROJECT_DESCRIPTION.md (1)
1-3:⚠️ Potential issue | 🟡 MinorDocument title and description still reference
mpt-tool.Lines 1 and 3 still use the old name (
mpt-tool CLI,mpt-tool is a command-line utility…), while every CLI invocation in the rest of the document now readsmpt-service-cli. This creates a confusing mix of names for the reader. Consider updating these tompt-service-clifor consistency with the rest of the document.📝 Suggested fix
-# mpt-tool CLI +# mpt-service-cli -mpt-tool is a command-line utility to scaffold, run, and audit migrations for MPT extensions. +mpt-service-cli is a command-line utility to scaffold, run, and audit migrations for MPT extensions.
albertsola
left a comment
There was a problem hiding this comment.
I would personally would remove the word cli from the command name. mpt-service is explicit enough given you will call that from bash/zsh



Closes MPT-17812
Changes
pyproject.tomlto rename the CLI command frommpt-tooltompt-service-cliMakefileto invoke the CLI using the newmpt-service-clicommand namedocs/PROJECT_DESCRIPTION.mdto reflect the new CLI command name in examples and instructions