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

EPS-786: Fixes: Assess code quality with PHP Insights #912

Open
wants to merge 21 commits into
base: php-insights-hfe
Choose a base branch
from

Conversation

vijetaR
Copy link
Contributor

@vijetaR vijetaR commented Oct 18, 2024

Description

Assess code quality with PHP Insights
Fixes done for php insights run

This pull request includes various changes to improve code quality, enhance functionality, and update documentation. The most important changes include adding a new Grunt task, updating the README file, and making several code improvements in the analytics classes.

Enhancements and new features:

  • Gruntfile.js: Added a new copy task to copy files while preserving file mode.

Documentation updates:

  • README.md: Updated the "Tested up to" and "Stable tag" versions, and added entries to the changelog for versions 1.6.44, 1.6.45, and 1.6.45.1. [1] [2]

Code improvements and refactoring:

Screenshots

Types of changes

Bug fix (non-breaking change which fixes an issue)

How has this been tested?

Checklist:

  • My code is tested
  • My code passes the PHPCS tests
  • My code follows accessibility standards
  • My code has proper inline documentation
  • I've included any necessary tests
  • I've included developer documentation
  • I've added proper labels to this pull request

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