Skip to content

Conversation

@sreichel
Copy link
Contributor

@sreichel sreichel commented Aug 31, 2023

Description (*)

Added PHPMD to worklow.

Related Pull Requests

  1. See PHPMD: added basic config #2771

Manual testing scenarios (*)

  1. ddev phpmd <path>

Questions or comments

Contribution checklist (*)

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All automated tests passed successfully (all builds are green)
  • Add yourself to contributors list

@sreichel sreichel changed the title Phpmd Add PHPMD to workflow Aug 31, 2023
@addison74
Copy link
Contributor

@sreichel - Nice to see you creating PRs. Please note there are 1225 warnings reported by PHPMD. Is it OK?

@sreichel sreichel closed this Sep 1, 2023
@OpenMage OpenMage locked as too heated and limited conversation to collaborators Sep 1, 2023
@OpenMage OpenMage deleted a comment from addison74 Sep 1, 2023
@OpenMage OpenMage deleted a comment from addison74 Sep 1, 2023
@OpenMage OpenMage deleted a comment from Flyingmana Sep 1, 2023
@OpenMage OpenMage deleted a comment from sreichel Sep 11, 2023
@OpenMage OpenMage deleted a comment from fballiano Sep 11, 2023
@OpenMage OpenMage deleted a comment from fballiano Sep 11, 2023
@OpenMage OpenMage deleted a comment from fballiano Sep 11, 2023
@OpenMage OpenMage deleted a comment from fballiano Sep 11, 2023
@OpenMage OpenMage deleted a comment from sreichel Sep 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants