Skip to content

Rename package and PHP namespace to ShadeTheory - #136

Open
shadetheory wants to merge 2 commits into
masterfrom
cursor/rename-namespace-shadetheory-1698
Open

shadetheory wants to merge 2 commits into
masterfrom
cursor/rename-namespace-shadetheory-1698

Conversation

@shadetheory

@shadetheory shadetheory commented Nov 12, 2026 •

Copy link
Copy Markdown
Owner

Summary

Breaking rename now that the jordanpartridge GitHub account is gone and the org is shadetheory. The repository slug stays github-client.

  • Composer package is shadetheory/github-client. PSR-4 autoload, autoload-dev, the Laravel provider, and the Github facade alias use ShadeTheory\GithubClient.
  • PHP namespaces, use statements, and string class names follow that root. Docs, README examples, tests, issue-template links, the funding handle, the license handle, and the release post package name use shadetheory.
  • Config publish tag stays github-client-config (Spatie package name is still github-client). Author name "Jordan Partridge" is unchanged. Past changelog entries are unchanged. Unreleased notes the break and the new composer require.

CI follow-up

  • Roave compares this branch to the latest release tag, which still has JordanPartridge\GithubClient. The baseline ignores those class deletions and does not match ShadeTheory\GithubClient.
  • .agents/ is export-ignore, so the dist archive stays lean. Local lean-package-validator v6.3.2 reports the archive as lean.

Tests

  • composer dump-autoload
  • Pest: 251 passed (907 assertions)
  • Pint --test: passed
  • PHPStan (phpstan.neon.dist, 512M): no errors

rg -i jordanpartridge (excluding vendor) hits CHANGELOG.md and the Roave baseline. The changelog hits are the new breaking note plus historical v0.2.1 / v0.2 / 0.1a PR links. The baseline names the old namespace so those deletions stay ignored until a release tag contains the new one.

Open in Web Open in Cursor 

The GitHub account is gone and the org is shadetheory, so the Composer package, PSR-4 root, and Laravel discovery entries move with it.

Co-authored-by: Jordan Partridge <jordan@partridge.rocks>
@shadetheory shadetheory added the risk:high Needs a human review before merge label Nov 12, 2026
Baseline the old JordanPartridge classes in Roave, and keep the .agents setup scripts out of the distribution archive.

Co-authored-by: Jordan Partridge <jordan@partridge.rocks>
@shadetheory

Copy link
Copy Markdown
Owner Author

Quality Gate: failure

Found 1 violation(s) across 6 tool(s).

Quality Check Results

pint: 0 violations, pest: 0 violations, phpstan: 0 violations, rector: 0 violations, lean-package: 1 violations, security-audit: 0 violations

Review the individual run outputs for detailed fix suggestions.

@shadetheory

Copy link
Copy Markdown
Owner Author

Quality Gate: success

All quality checks passed.

Quality Check Results

All tools passed without violations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk:high Needs a human review before merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants