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

Fix: Linting issues and Update readme.md file #104

Merged
merged 10 commits into from
Feb 12, 2025

Conversation

nani-samireddy
Copy link
Member

@nani-samireddy nani-samireddy commented Feb 11, 2025

Description

This PR fixes the eslint errors in JS and PHP files and updates the readme.md file with the new npm scripts

Technical Details

  • Linting errors such as spaces around parenthesis, comments for internal dependencies
  • undefined variable PHPCS error in the comments.php
  • Update readme.md file with new build scripts

Checklist

Screenshots

To-do

Fixes/Covers issue

Fixes #

@nani-samireddy nani-samireddy changed the base branch from master to enhancement/update-npm-scripts-to-use-wp-scripts February 11, 2025 13:05
@nani-samireddy nani-samireddy marked this pull request as ready for review February 12, 2025 06:27
@nani-samireddy nani-samireddy changed the base branch from enhancement/update-npm-scripts-to-use-wp-scripts to master February 12, 2025 06:38
@nani-samireddy nani-samireddy changed the title Fix/php and js linting issues Fix: Linting issues and Update readme.md file Feb 12, 2025
@divyarajmasani divyarajmasani merged commit 4035533 into master Feb 12, 2025
2 checks passed
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.

2 participants