Skip to content

docs: modernize CONTRIBUTING.md for plugin scope#110

Merged
JohnMcLear merged 3 commits into
mainfrom
chore/modernize-docs-a11y-helpers
May 9, 2026
Merged

docs: modernize CONTRIBUTING.md for plugin scope#110
JohnMcLear merged 3 commits into
mainfrom
chore/modernize-docs-a11y-helpers

Conversation

@JohnMcLear
Copy link
Copy Markdown
Member

Summary

Part of the fleet-wide plugin docs modernization (pilot: ether/ep_align#182).

  • Rewrite CONTRIBUTING.md for plugin scope (target branch main, plugin install via pnpm run plugins i --path, i18n + a11y are standing requirements). Cross-links to ether/etherpad's CONTRIBUTING.md for shared rules.

Out of scope (separate campaigns)

  • AGENTS.md — Phase 2.
  • a11y/i18n audit and fixes — Phase 3.
  • ep_plugin_helpers adoption audit — Phase 4.

Replaces the etherpad-lite-era boilerplate with a plugin-specific
guide: target branch is main, local install via
pnpm run plugins i --path, tests run via the etherpad harness,
and i18n/a11y are now standing requirements.
@qodo-code-review
Copy link
Copy Markdown

ⓘ You've reached your Qodo monthly free-tier limit. Reviews pause until next month — upgrade your plan to continue now, or link your paid account if you already have one.

JohnMcLear added 2 commits May 9, 2026 17:48
ether/etherpad-lite's package.json declares packageManager: pnpm@11.0.6.
pnpm/action-setup@v6 errored with ERR_PNPM_BAD_PM_VERSION when version
was pinned to 10 because the action saw the conflict with the core's
packageManager field. Bumping the pin to 11 resolves the mismatch.
Pinning pnpm version on the action while ether/etherpad-lite's
package.json has "packageManager": "pnpm@11.0.6" triggers
ERR_PNPM_BAD_PM_VERSION regardless of the pinned value. Drop the
version pin and point the action at the core's package.json so it
reads the canonical version from packageManager.
@JohnMcLear JohnMcLear merged commit 16a8426 into main May 9, 2026
3 checks passed
@JohnMcLear JohnMcLear deleted the chore/modernize-docs-a11y-helpers branch May 9, 2026 17:25
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