Replace unnecessary PHP comment with HTML comment in customize.php - #2
Replace unnecessary PHP comment with HTML comment in customize.php#2tewariharish wants to merge 1 commit into
Conversation
In `src/wp-admin/customize.php` (around line 227 and 266), there is a PHP opening tag used solely for an inline comment: <ul class="customize-pane-parent"><?php // Panels and sections are managed here via JavaScript ?></ul> Since no PHP code is being executed, the PHP tag is unnecessary. This can be replaced with an HTML comment to improve code readability and maintainability. WordPress coding standards generally prefer avoiding unnecessary PHP tags when no PHP code is being executed
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughIn ChangesCustomizer Placeholder Comment Syntax
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
https://core.trac.wordpress.org/ticket/65497
In
src/wp-admin/customize.php(around line 227 and 266), there is a PHP opening tag used solely for an inline comment:Since no PHP code is being executed, the PHP tag is unnecessary. This can be replaced with an HTML comment to improve code readability and maintainability.
WordPress coding standards generally prefer avoiding unnecessary PHP tags when no PHP code is being executed
Trac ticket:
Use of AI Tools
This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.
Summary by CodeRabbit