Skip to content

chore: share one .editorconfig across plugins - #843

Merged
josephfusco merged 2 commits into
mainfrom
chore/shared-plugin-editorconfig
Oct 26, 2026
Merged

josephfusco merged 2 commits into
mainfrom
chore/shared-plugin-editorconfig

Conversation

@josephfusco

Copy link
Copy Markdown
Member

Replaces the five per-plugin .editorconfig files with one at plugins/.editorconfig. Editors look for .editorconfig in parent folders, so every plugin, and hwp-cli, which had none, picks it up. Nothing is generated or copied.

Which settings

The copies came in two variants:

  • hwp-documents, hwp-previews, wpgraphql-logging: tabs for everything. That includes YAML, which doesn't allow tab indentation.
  • wpgraphql-debug-extensions, wpgraphql-webhooks: tabs by default, 2-space Markdown and YAML, tabs for JSON, and no whitespace trimming in .txt files.

The shared file uses the second variant. No plugin YAML file is tab-indented today, so it matches how the files are already written.

Scope

It lives in plugins/, not the repo root, so the example apps under examples/ and packages/ keep their own styles. Each plugin's old .editorconfig was already root = true and covered its own examples/ folder, so nothing inside the plugins changes scope. Zips don't change either, because .editorconfig is excluded from them.

Use of AI Tools

AI assistance: Yes
Tool(s): Claude Code
Model(s): Claude Opus 5.5
Used for: Assisting with consolidating duplicated editor config.

The five plugin copies came in two variants. plugins/.editorconfig keeps the fuller one,
which also sets 2-space YAML (the other variant tab-indented YAML, which YAML doesn't
allow). EditorConfig applies it to every plugin folder without a build step.
@josephfusco
josephfusco requested review from a team and colinmurphy as code owners October 24, 2026 02:15
@josephfusco josephfusco added [Type] Task Maintenance, upkeep, and other non-functional work [Area] Infrastructure Issues for the repository infrastructure labels Oct 24, 2026
@changeset-bot

changeset-bot Bot commented Oct 24, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 1535b72

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions

github-actions Bot commented Oct 24, 2026 •

Copy link
Copy Markdown
Contributor

🔍 WordPress Plugin Check Report

❌ Status: Failed

📊 Report

🎯 Total Issues ❌ Errors ⚠️ Warnings
1 1 0

❌ Errors (1)

📁 readme.txt (1 error)
📍 Line 🔖 Check 💬 Message
0 outdated_tested_upto_header Tested up to: 6.9 < 7.1. 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

@josephfusco
josephfusco merged commit 075d5d7 into main Oct 26, 2026
4 checks passed
@josephfusco
josephfusco deleted the chore/shared-plugin-editorconfig branch October 26, 2026 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Area] Infrastructure Issues for the repository infrastructure [Type] Task Maintenance, upkeep, and other non-functional work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant