Skip to content

Conversation

@Philipp91
Copy link
Contributor

It's the latest PHP CS Fixer default: https://cs.symfony.com/doc/rules/phpdoc/no_superfluous_phpdoc_tags.html

The reasoning that it's not useful for any IDEs or for PHP doc generation: https://www.drupal.org/project/coding_standards/issues/3335320

It's the latest PHP CS Fixer default: https://cs.symfony.com/doc/rules/phpdoc/no_superfluous_phpdoc_tags.html

The reasoning that it's not useful for any IDEs or for PHP doc generation: https://www.drupal.org/project/coding_standards/issues/3335320
@ampaze
Copy link
Contributor

ampaze commented Oct 14, 2025

Housekeeping is cool, but would it be possible to prioritize VoP?

If you don't plan on working on it, that is ok too. I just need to know, in this case I will do another implementation.

@Philipp91
Copy link
Contributor Author

I am planning to work on it. It's just I can't work when my IDE shows all red and composer cs-fix touches files left and right. Ideally, everyone would do this continuously and GitHub would flag PRs that don't abide by this, but that appears to have slipped.

@ampaze
Copy link
Contributor

ampaze commented Oct 14, 2025

My apologies.

@nemiah nemiah merged commit 0720da4 into nemiah:master Oct 14, 2025
5 checks passed
@Philipp91
Copy link
Contributor Author

My apologies.

No worries, not at all our fault. I believe the PHP CS Fixer enforcement got accidentally dropped during the Travis -> GitHub Actions migration. I'll take care of this after VoP.

@Philipp91 Philipp91 deleted the inheritdoc-remove branch October 18, 2025 11:56
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