Skip to content

feat: P002827-13 implement Drupal - #72

Open
rutgerrademaker wants to merge 6 commits into
masterfrom
feature/add-drupal-support
Open

feat: P002827-13 implement Drupal#72
rutgerrademaker wants to merge 6 commits into
masterfrom
feature/add-drupal-support

Conversation

@rutgerrademaker

Copy link
Copy Markdown
Contributor

No description provided.

- /^pdepend.xml$/

yamllint.parse_constant: true
yamllint.parse_custom_tags: false

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why is this removed, it is added for a reason, see PR #69

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks for flagging, guess I made a mistake here, had no intentions to change things
Also leaving out the "magento" related changes, as we only target drupal support in this PR
cc @leonhelmus

@rutgerrademaker
rutgerrademaker force-pushed the feature/add-drupal-support branch 2 times, most recently from fabec2d to 69a9347 Compare July 24, 2026 08:43
After finding out PHPCS was always passing in CI, it turned
out that this is related to the <root> folder in CI being
named . As  was marked as excluded in phpcs.xml,
PHPCS was always passing in CI. Grumphp seems to offer
better support for only excluding the  directory
inside the <root> directory.
@rutgerrademaker
rutgerrademaker force-pushed the feature/add-drupal-support branch from 69a9347 to 659a113 Compare July 24, 2026 11:54
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.

3 participants