| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
There was a problem hiding this comment.
This PR decouples ValueObject from tiny-blocks/immutable-object and tightens equals() semantics by requiring same-class comparison and strict property equality, along with related documentation and tooling updates.
Changes:
Copilot reviewed 17 out of 17 changed files in this pull request and generated 7 comments.
Show a summary per file| File | Description |
|---|---|
| src/ValueObjectBehavior.php | Enforces same-class + strict get_object_vars() equality. |
| src/ValueObject.php | Removes Immutable extension from the public interface. |
| tests/ValueObjectTest.php | Adjusts equality test cases to match stricter semantics. |
| README.md | Updates guidance and adds FAQ describing strict equality behavior. |
| composer.json | Removes immutable-object dependency; refreshes metadata/dev tooling. |
| infection.json.dist | Changes mutator configuration. |
| .github/workflows/ci.yml | Updates artifact action versions; removes bcmath extension. |
| Makefile | Adds composer normalize to configure; adds show-outdated target. |
| .gitattributes | Adds normalization/diff driver settings and updates export-ignore list. |
| .editorconfig | Adds editor formatting defaults. |
| .github/copilot-instructions.md | Adds Copilot guidance referencing .claude/ rules. |
| .claude/** | Introduces repo-wide rules for modeling, testing, documentation, and workflows. |
src/ValueObject.php:22
/**
* Compares this ValueObject with another to determine if they are equal.
* Two ValueObjects are considered equal if their properties have the same values.
*
* @param ValueObject $other The ValueObject to compare with.
* @return bool True if the objects are equal, false otherwise.
*/
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
No description provided.