Renamed the package from 'drevops/tui' to 'drevops/phptui'. - #155
Merged
Conversation
|
Important Review skippedToo many files! This PR contains 516 files, which is 216 over the limit of 300. To get a review, reduce the PR to 300 files or fewer by splitting it into smaller PRs or changing its base branch. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (7)
📒 Files selected for processing (516)
You can disable this status message by setting the Comment |
|
🚀 Deployed on https://6a7bd2dc724962e15826895e--tui-docs.netlify.app |
|
AlexSkrypnyk
added
the
Needs review
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #155 +/- ##
=======================================
Coverage 98.05% 98.05%
=======================================
Files 150 150
Lines 6054 6054
=======================================
Hits 5936 5936
Misses 118 118 ☔ View full report in Codecov by Harness. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Renames the package from
drevops/tuitodrevops/phptuiacross its whole identity: the Composer package name, the PHP namespace, the default headless environment prefix, the repository URLs and the documentation site configuration. The library already published its homepage asphptui.devwhile shipping under thetuiname, so this closes the gap between what the project is called and what it ships as.This is a breaking change for consumers on two axes: every
use DrevOps\Tui\...import becomesuse DrevOps\PhpTui\..., and every headlessTUI_<ID>environment variable becomesPHPTUI_<ID>. The facade class staysTui, sonew Tui($form)andTui::collect()are unchanged.Changes
Package identity
composer.json: the package name, both support URLs, and the PSR-4 roots forsrc/andtests/phpunit.README.md: the six shields badges, thecomposer requireline, the logo alt text and the prose references.CONTRIBUTING.md,playground/README.mdanddocs/architecture/README.md.PHP namespace
DrevOps\Tui\becomesDrevOps\PhpTui\acrosssrc/,tests/,playground/,translations/and every documentation code block - namespace declarations,usestatements, docblock FQNs and@packagetags alike.@param,@returnand@vartag lines are left long, matching the convention the codebase already follows for unbreakable type expressions.Environment prefix
TUI_becomesPHPTUI_inTui::__construct(), and every example, test and playground usage block that names it. Forms that declare their own prefix with->envPrefix()are unaffected.Documentation site
docusaurus.config.js:projectName, the site title, the navbar logo alt text, theeditUrland the three GitHub links;phptuiadded to the search keywords.docs/package.jsonanddocs/package-lock.json: the package is now@drevops/phptui-docs.docs/src/pages/index.js: the install command, the repository blob base and the terminal chrome label on the landing page.docs/cspell.json:PHPTUIandphptuiadded to the dictionary so the spellcheck gate passes.Architecture diagrams
render-phptui-diagramsskill rather than hand-patched.Agent tooling
render-tui-diagramsskill is renamed torender-phptui-diagrams, following the project-scope segment of the skill naming convention, along with its references inAGENTS.mdand the docs.Verification
composer test- 2918 tests, 6445 assertions, all passing.composer lint- PHPCS across 465 files, PHPStan level 9, Rector, all clean.php docs/util/audit-svgs.php- 661 assets audited, no issues.npm run buildandnpm testindocs/- the site builds, and jest, cspell and Prettier all pass.PHPTUI_NAME='Weekly Box' php playground/01-quickstart.php < /dev/nullresolves the answer through the new prefix end to end.Follow-up outside this PR
drevops/phptui(GitHub redirects the old URLs, so the badges in this diff resolve either way).tui-docs.netlify.apppreview URL inREADME.mdis left as-is, since renaming that site is a separate manual step.Before / After