Skip to content

Theme Directory: rely on the automated scan result for child theme uploads - #854

Closed
obenland wants to merge 1 commit into
WordPress:trunkfrom
obenland:update/child-theme-scan
Closed

Theme Directory: rely on the automated scan result for child theme uploads#854
obenland wants to merge 1 commit into
WordPress:trunkfrom
obenland:update/child-theme-scan

Conversation

@obenland

@obenland obenland commented Sep 1, 2026

Copy link
Copy Markdown
Member

Child themes now pass the automated Theme Check scan on their own — the recent plugin updates skip only the requirements a parent provides (base template functions, index.php, title-tag support) while still scanning the child's own files.

Because of that, the upload handler no longer needs to special-case child themes in check_theme(). This removes that special-casing so a child theme's scan result is applied the same way as for any other theme.

…loads

Recent Theme Check updates let child themes pass the automated scan on their
own, skipping only the requirements their parent provides. The upload handler
no longer needs to special-case child themes, so let their scan result apply
the same as for any other theme.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 1, 2026 16:11
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

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 props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props obenland.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Theme Directory upload flow to treat child theme uploads the same as any other theme by relying on the automated Theme Check scan result, instead of force-passing child themes regardless of scan outcome.

Changes:

  • Removed the child-theme-specific override in check_theme() that always returned a passing result when a parent theme exists.
  • Ensured the Theme Check scan verdict is applied consistently to both parent and child themes during uploads.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@bazza bazza closed this in b7dbea2 Sep 1, 2026
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.

2 participants