Skip to content

T71 config show - #73

Merged
niden merged 3 commits into
masterfrom
T71-config-show
Oct 31, 2026
Merged

niden merged 3 commits into
masterfrom
T71-config-show

Conversation

@niden

@niden niden commented Oct 31, 2026

Copy link
Copy Markdown
Member

Hello!

In raising this pull request, I confirm the following:

  • I have checked that another pull request for this purpose does not exist
  • I wrote some tests for this PR
  • composer test, composer cs and composer analyze all pass
  • I have created a PR for the documentation about this change, if user facing

  • Added the models key to config:show in an MVC project: a camelCase row with its origin, and a TABLE CLASS table with each line of models.classes, sorted by table. A wrong key gives one models row with the error; config:show still exits with 0, and make:model stops on the error. Added Crest\Generator\ModelNames::camelCase().
  • Added a Note line to make:model for a table whose class comes from models.classes: it names the old model and base, which crest keeps, and the related tables to run make:model for again.

Thanks

niden added 3 commits October 31, 2026 12:38
Assisted-by: Claude Code
Assisted-by: Claude Code
Assisted-by: Claude Code
@niden
niden requested a review from Jeckerson October 31, 2026 17:45
@niden niden self-assigned this Oct 31, 2026
@niden niden added the enhancement New feature or request label Oct 31, 2026
@github-actions

This comment has been minimized.

@github-actions

Copy link
Copy Markdown

Code Metrics Report

master (8410e8c) #73 (861ba26) +/-
Coverage 99.7% 99.7% +0.0%
Test Execution Time 11s 10s -1s
Details
  |                     | master (8410e8c) | #73 (861ba26) |  +/-  |
  |---------------------|------------------|---------------|-------|
+ | Coverage            |            99.7% |         99.7% | +0.0% |
  |   Files             |               86 |            86 |     0 |
  |   Lines             |             3046 |          3095 |   +49 |
+ |   Covered           |             3039 |          3088 |   +49 |
+ | Test Execution Time |              11s |           10s |   -1s |

Code coverage of files in pull request scope (100.0% → 100.0%, patch 100.0%)

Files Coverage +/- Patch Coverage Status
src/Command/Config/ShowCommand.php 100.0% 0.0% 100.0% modified
src/Command/Make/ModelCommand.php 100.0% 0.0% 100.0% modified
src/Generator/ModelNames.php 100.0% 0.0% 100.0% modified
src/Project/Config.php 100.0% 0.0% - modified

Reported by octocov

@niden
niden merged commit ee03baa into master Oct 31, 2026
40 checks passed
@niden
niden deleted the T71-config-show branch October 31, 2026 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant