Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use PHPMD for code validation #8

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

Moustafa-Elgammal
Copy link

@Moustafa-Elgammal Moustafa-Elgammal commented Jun 2, 2024

This PR includes:

Screenshot 2024-06-02 at 14 20 07

fix MissingImport warning
…values to variables in if clauses and the like ..."
fix  BooleanArgumentFlag     The

- [x] method getText has a boolean flag argument $without_cmd, which is a certain sign of a Single Responsibility Principle violation.

- [x] The method printError has a boolean flag argument $return, which is a certain sign of a Single Responsibility Principle violation.

- [x] The method printError has a boolean flag argument $return, which is a certain sign of a Single Responsibility Principle violation.
…future expanding and handle bypassing the current status.
@Moustafa-Elgammal Moustafa-Elgammal force-pushed the feature/use-php-mess-detect branch from 35ab149 to 9c47d9e Compare June 30, 2024 01:48
@Moustafa-Elgammal
Copy link
Author

any update @shahradelahi

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant