Skip to content

chore: update repo references to create-wp-plugin-cli - #2

Merged
akshat009 merged 1 commit into
mainfrom
chore/rename-cli-refs
Aug 19, 2026
Merged

chore: update repo references to create-wp-plugin-cli#2
akshat009 merged 1 commit into
mainfrom
chore/rename-cli-refs

Conversation

@akshat009

Copy link
Copy Markdown
Owner
  • package.json: repository.url, homepage, bugs.url point to renamed GitHub repo
  • README.md: fix npx github: fallback command to use new repo name
  • index.js: update welcome banner text
  • package-lock.json: sync name/bin fields via npm install

- package.json: repository.url, homepage, bugs.url point to renamed GitHub repo
- README.md: fix npx github: fallback command to use new repo name
- index.js: update welcome banner text
- package-lock.json: sync name/bin fields via npm install
@akshat009
akshat009 merged commit 9eb7a10 into main Aug 19, 2026
2 checks passed
@akshat009
akshat009 deleted the chore/rename-cli-refs branch August 19, 2026 07:43
akshat009 added a commit that referenced this pull request Aug 29, 2026
…repare-dist (#11, #12 revised)

Re-audit #2 retracted "drop the */templates/* exclude" — the WC email/My-Account
overrides genuinely fail three sniffs (PrefixAllGlobals on core WC hook names +
loosely-named locals; EscapeOutput on plain-text bodies that use
wp_strip_all_tags). But the blanket path exclude also silenced EscapeOutput on
the HTML email template, where escaping does matter.

Now: when a woo:email / woo:my-account module ships a templates/ dir,
phpcs.xml adds `<file>./templates</file>` and excludes only:
  - WordPress.NamingConventions.PrefixAllGlobals for */templates/*
  - WordPress.Security.EscapeOutput for */templates/emails/plain/* only
Everything else in templates/ (docblocks, spacing, i18n, the HTML email's
escaping) is linted. Gated on a new has_wc_template_overrides flag; a scaffold
without those modules has no templates/ reference at all.

Also: `composer prepare-dist` = `composer install --no-dev --optimize-autoloader`,
so `npm run plugin-zip` doesn't depend on the user reading the release docs.
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