Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow output to file or remove maintainers message #270

Closed
jorgetutor opened this issue Apr 14, 2022 · 2 comments
Closed

Allow output to file or remove maintainers message #270

jorgetutor opened this issue Apr 14, 2022 · 2 comments

Comments

@jorgetutor
Copy link

How is drupal-check installed?

drupal-check is installed as a dependency to my project

Environment:

  • OS: Linux
  • PHP Version: 7.2
  • Drupal core: 8.x.x

Describe the bug
When the output selected is checkstyle it is also retrieving module maintainers information, so not possible easy to parse the output and save it into a file.

Alternatives:

  • Do not display the maintainer's information
  • Allow a --output option to save to a file. Maybe is already there but I have not seen it.

Console output

vendor/bin/drupal-check docroot/modules/custom --format=checkstyle --no-progress'
<?xml version="1.0" encoding="UTF-8"?>
<checkstyle>
<file name="web/modules/custom/idn_celebrity_house/src/CelebrityHouseListBuilder.php">
  <error line="68" column="1" severity="error" message="Missing explicit access check on entity query." />
</file>
<file name="web/modules/custom/idn_self_promotion/src/SelfPromotionListBuilder.php">
  <error line="68" column="1" severity="error" message="Missing explicit access check on entity query." />
</file>
</checkstyle>
Thanks for using drupal-check!

Consider sponsoring the development of the maintainers which make drupal-check possible:

- phpstan (ondrejmirtes): https://github.com/sponsors/ondrejmirtes
- phpstan-deprecation-rules (ondrejmirtes)): https://github.com/sponsors/ondrejmirtes
- phpstan-drupal (mglaman)): https://github.com/sponsors/mglaman
- drupal-check (mglaman)): https://github.com/sponsors/mglaman
@mglaman
Copy link
Owner

mglaman commented Apr 27, 2022

Ah shoot, I didn't realize it broke that.

@mglaman
Copy link
Owner

mglaman commented Apr 29, 2022

@jorgetutor fixing in a release soon. I recommend moving to PHPStan directly. Here are the docs: https://www.drupal.org/docs/develop/development-tools/phpstan/getting-started

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

No branches or pull requests

2 participants