Skip to content

chore: require validators 0.6 - #72

Merged
ChiragAgg5k merged 4 commits into
mainfrom
chore/validators-0.6
Aug 27, 2026
Merged

chore: require validators 0.6#72
ChiragAgg5k merged 4 commits into
mainfrom
chore/validators-0.6

Conversation

@ChiragAgg5k

@ChiragAgg5k ChiragAgg5k commented Aug 27, 2026

Copy link
Copy Markdown
Member

Summary

  • narrow the validators dependency from 0.* to ^0.6
  • raise the minimum PHP version to 8.5, matching validators 0.6
  • update the lock file to validators 0.6.0 and the CI matrix

Validation

  • Composer dependency resolution
  • Pint
  • PHPStan level 4
  • unit tests pass; registrar integration tests require provider credentials

@ChiragAgg5k
ChiragAgg5k force-pushed the chore/validators-0.6 branch from 98cfc4b to 94b32c1 Compare August 27, 2026 05:41
@greptile-apps

greptile-apps Bot commented Aug 27, 2026

Copy link
Copy Markdown

Greptile Summary

The PR updates the validators dependency and minimum supported PHP version, aligns CI and documentation with PHP 8.5, and extends Name.com transfer-error handling.

  • Requires utopia-php/validators 0.6 and PHP 8.5.
  • Updates the lock file and CI matrix for PHP 8.5.
  • Corrects the documented PHP requirement.
  • Adds a typed Name.com transfer-error mapping and improves test diagnostics.

Confidence Score: 5/5

The PR appears safe to merge because no blocking failure remains within the follow-up review scope.

The previously reported PHP requirement mismatch is fixed: README.md now advertises PHP 8.5 or later, matching composer.json.

Important Files Changed

Filename Overview
composer.json Raises the PHP requirement to 8.5 and narrows the validators dependency to the 0.6 release line.
composer.lock Locks validators 0.6.0 and synchronizes the Composer platform requirement with PHP 8.5.
README.md Fixes the previously reported requirements mismatch by documenting PHP 8.5 or later.
.github/workflows/test.yml Removes PHP 8.4 from the test matrix to match the new package requirement.
src/Domains/Registrar/Adapter/NameCom.php Adds handling intended to classify a missing transfer domain as not transferable.
tests/Registrar/Base.php Expands transfer-test failure diagnostics with the exception message and code.

Reviews (4): Last reviewed commit: "fix: classify missing domains as not tra..." | Re-trigger Greptile

Comment thread composer.json
@ChiragAgg5k
ChiragAgg5k merged commit 021ce3c into main Aug 27, 2026
4 checks passed
@ChiragAgg5k
ChiragAgg5k deleted the chore/validators-0.6 branch August 27, 2026 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants