Skip to content

Conversation

@amitintelegencia
Copy link

Fix: Resolve WP VIP PHPCS errors flagged by the WPVIP GitHub bot.

This update fixes several coding standard issues reported by the WPVIP PHPCS bot, improving code quality and ensuring full compliance with WPVIP coding requirements. No functional changes were made to business logic — all updates are related to formatting, escaping, validation, and sanitization as required by WPVIP rules.

For reference, the WPVIP PHPCS ruleset can be found here:
https://github.com/Automattic/VIP-Coding-Standards

@enricobattocchi
Copy link
Member

Hey @amitintelegencia, can you share the error report from the VIP bot? it's not clear to us why it's flagging those attributes.

@manishshukla76
Copy link

Hey @enricobattocchi
Please find the error report from the VIP Bot.
phpcs-report-yoast-seo.csv

Thanks

@enricobattocchi
Copy link
Member

Thanks @manishshukla76, but I could not find any mention of SVG checks flagging the focusable attribute in that report... do you have the report of the SVG analysis instead of the PHPCS check? From what I see here it's a different scanner

@manishshukla76
Copy link

@enricobattocchi
Please find the desired report.
yoast-seo-errors.pdf

Thanks

@enricobattocchi
Copy link
Member

Thanks @manishshukla76! I'll have a look

@github-actions
Copy link

A merge conflict has been detected for the proposed code changes in this PR. Please resolve the conflict by either rebasing the PR or merging in changes from the base branch.

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.

4 participants