valkyrjaio / valkyrja-starter-app-php / 25573026960

0%

push

github

[Cli] Add Header banner message class and update all commands to use it (#46)

# Description

Removes the now-redundant `App\Cli\Constant\CliHeader` and
`App\Cli\Message\AppCliHeader` classes, replacing them with the
framework's new `Valkyrja\Cli\Interaction\Message\Header` directly in
`TestCommand`.

---

## Types of Changes

- [x] Improvement _(non-breaking change which improves code)_
- [ ] Bug fix _(non-breaking change which fixes an issue)_
- [ ] New feature _(non-breaking change which adds functionality)_
- [ ] Deprecation _(breaking change which removes functionality)_
- [ ] Breaking change _(fix or feature that would cause existing
functionality to change)_
- [ ] Documentation improvement

---

## Changes

- **`App\Cli\Constant\CliHeader`** — deleted; superseded by the
framework `Header`
- **`App\Cli\Message\AppCliHeader`** — deleted; superseded by the
framework `Header`
- **`TestCommand`** — uses `new Header($this->config->namespace,
$this->config->version, $route)` directly

---------

Co-authored-by: valkyrja-volundr[bot] <249390750+valkyrja-volundr[bot]@users.noreply.github.com>
Co-authored-by: MelechMizrachi <1179171+MelechMizrachi@users.noreply.github.com>

0 of 9 new or added lines in 3 files covered. (0.0%)

0 of 293 relevant lines covered (0.0%)

0.0 hits per line