Skip to content

Modernize coding standards configuration#15

Open
jeremyfelt wants to merge 1 commit into
trunkfrom
update/coding-standards-config
Open

Modernize coding standards configuration#15
jeremyfelt wants to merge 1 commit into
trunkfrom
update/coding-standards-config

Conversation

@jeremyfelt
Copy link
Copy Markdown
Member

Summary

  • Switch from raw WPCS/PHPCompat dependencies to happyprime/coding-standards
  • Simplify phpcs.xml to reference the shared Happy Prime ruleset
  • Add phpstan.neon.dist (level 9) with phpstan script
  • Add .editorconfig for consistent editor formatting
  • Add phpstan.yml, deploy-to-readme.yml, deploy-to-wp.yml, and plugin-check.yml workflows
  • Update .gitignore and .distignore
  • Update composer.lock

Test plan

  • Verify composer phpcs still works (ruleset changed)
  • Verify composer phpstan runs successfully
  • Verify new workflows run on this PR

🤖 Generated with Claude Code

Switch from raw WPCS to happyprime/coding-standards, add phpstan,
add .editorconfig, add deploy and plugin-check workflows,
and update .gitignore and .distignore.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

🔍 WordPress Plugin Check Report

❌ Status: Failed

📊 Report

🎯 Total Issues ❌ Errors ⚠️ Warnings
4 4 0

❌ Errors (4)

📁 plugin.php (3 errors)
📍 Line 🔖 Check 💬 Message
0 plugin_header_no_license Missing "License" in Plugin Header. Please update your Plugin Header with a valid GPLv2 (or later) compatible license.
0 missing_direct_file_access_protection PHP file should prevent direct access. Add a check like: if ( ! defined( 'ABSPATH' ) ) exit;
79 WordPress.WP.I18n.MissingArgDomain Missing $domain parameter in function call to __().
📁 readme.txt (1 error)
📍 Line 🔖 Check 💬 Message
0 outdated_tested_upto_header Tested up to: 6.2 < 6.9. The "Tested up to" value in your plugin is not set to the current version of WordPress. This means your plugin will not show up in searches, as we require plugins to be compatible and documented as tested up to the most recent version of WordPress.

🤖 Generated by WordPress Plugin Check Action • Learn more about Plugin Check

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