Skip to content

Conversation

@rez1dent3
Copy link
Member

@rez1dent3 rez1dent3 commented Nov 17, 2025

  • Removed redundant PHPDoc comments in the Confirmable and ProductLimitedInterface interfaces.
  • Updated PHPDoc comments to include type hints for return values in various DTOs and services.
  • Enhanced type safety by specifying generic types in relationships within the Wallet model and related traits.
  • Cleaned up unused comments and improved clarity in method signatures across several classes.
  • Ensured consistency in type declarations and added assertions where necessary for better code quality.

…faces and models

- Removed redundant PHPDoc comments in the Confirmable and ProductLimitedInterface interfaces.
- Updated PHPDoc comments to include type hints for return values in various DTOs and services.
- Enhanced type safety by specifying generic types in relationships within the Wallet model and related traits.
- Cleaned up unused comments and improved clarity in method signatures across several classes.
- Ensured consistency in type declarations and added assertions where necessary for better code quality.
@rez1dent3 rez1dent3 changed the title Refactor PHPDoc comments and improve type hints across multiple inter… Refactor PHPDoc comments and improve type hints across multiple interfaces and models Nov 17, 2025
- Simplified the conversion of the available balance to a string by removing the conditional check for string type.
- Ensured consistent type casting for better clarity and maintainability.
- Added "deptrac/deptrac" as a development dependency in composer.json.
- Updated deptrac.yaml to use "classLike" collectors instead of "className" for better compatibility with the latest deptrac standards.
- Modified GitHub Actions workflow to remove the installation step for deptrac, as it is now included in composer dependencies.
@rez1dent3 rez1dent3 merged commit fbe3f19 into master Nov 17, 2025
101 checks passed
@rez1dent3 rez1dent3 deleted the phpstan-fixes branch November 17, 2025 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants