Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Github actions style and quality #43

Open
wants to merge 7 commits into
base: 3.x
Choose a base branch
from

Conversation

gpibarra
Copy link

No description provided.

Copy link

what-the-diff bot commented Apr 29, 2024

PR Summary

  • Addition of Multiple Configuration Files
    Numerous files were added to the repository which will contribute towards maintaining consistency in coding styles amongst developers, streamline tests, and manage dependencies.

  • Updations to .gitignore and resources/lang/fa/clauses.php files
    The .DS_Store and node_modules entries were removed from the .gitignore file to prevent storing unnecessary files in the repository. The resources/lang/fa/clauses.php file was updated to include a Farsi translation for the term "between".

  • Modifications in HasClauses.php file
    Apart from a new import, there were several changes in property declarations, function signatures, and methods to optimize the handling of logic and functionality.

  • Revisions to BooleanFilter.php, DateFilter.php, NumberFilter.php, TextFilter.php files
    These received updates in constants, methods and general syntax improvements. These changes enhance the way the program handles data, comparisons, and logical operations, thereby improving overall efficiency.

  • Inclusion of Test Files
    Files like ArchTest.php, ExampleTest.php, Pest.php, and TestCase.php were added to implement and manage tests effectively. This aids in promptly detecting and resolving potential code quality issues and bugs, solidifying the stability of the application.

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.

1 participant