Added common command to check direct use of dependencies, #PG-3272

[AltamashShaikh]

Description:

Added common command to check direct use of dependencies
Fixes: #PG-3272

Review

• Functional review done
• Potential edge cases thought about (behavior of the code with strange input, with strange internal state or possible interactions with other Matomo subsystems)
• Usability review done (is anything maybe unclear or think about anything that would cause people to reach out to support)
• Security review done
• Wording review done
• Code review done
• Tests were added if useful/possible
• Reviewed for breaking changes
• Developer changelog updated if needed
• Documentation added if needed
• Existing documentation updated if needed

[michalkleiner]

Generally I'm happy with this, any notes/concerns from @matomo-org/core-reviewers?

[michalkleiner]

Also, should this be mentioned in the changelog? I'm 50/50 on that.

[AltamashShaikh]

Also, should this be mentioned in the changelog? I'm 50/50 on that.

@michalkleiner We added in TestRunner as we don't need a release for it, so I don't think we would need to mention it in changelog, if we do might be useful for other plugin developers.

[michalkleiner]

@bx80 @caddoo any thoughts here?

[bx80]

This looks good to me, just a minor comment on the paramter descriptions 👍 Not sure that we need to mention the new command in the changelog as long as the developer documentation is updated.

[AltamashShaikh]

This looks good to me, just a minor comment on the paramter descriptions 👍 Not sure that we need to mention the new command in the changelog as long as the developer documentation is updated.

@bx80 Created matomo-org/developer-documentation#778 to mention the same in developer documentation.

Can we merge this PR now ?

[michalkleiner]

Thanks for opening the docs PR as well, @AltamashShaikh.