Skip to content

Conversation

@theodorosploumis
Copy link
Owner

Summary

  • add AI-targeted guidance files (AGENTS, rules, commands, Claude Code skills) derived from README sections 2 and 3
  • provide drush-based validation scripts by Site building and Theming subsections for Drupal 10+
  • document the new AI helper folder and usage

Testing

  • Not run (not requested)

Codex Task

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +8 to +12
drush php:eval '
use Drupal\system\Entity\Theme;
$fail = 0;
$themes = Theme::loadMultiple();
foreach ($themes as $theme) {

Choose a reason for hiding this comment

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

P1 Badge Theming validator uses non-existent Theme entity

The theming script imports Drupal\system\Entity\Theme and calls Theme::loadMultiple(), but Drupal themes are extensions exposed through the theme handler, not config entities. On Drupal 10 this use statement causes drush php:eval to fatally error (Class "Drupal\system\Entity\Theme" not found), so none of the validation logic can run. Please switch to the theme handler (e.g., \Drupal::service('theme_handler')->listInfo()) or another supported API so the script can execute.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants