Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Option to only report on file changes? #4

Closed
rsoesemann opened this issue Nov 29, 2021 · 1 comment
Closed

Option to only report on file changes? #4

rsoesemann opened this issue Nov 29, 2021 · 1 comment
Labels
question Further information is requested

Comments

@rsoesemann
Copy link
Member

Does this offical Github Action have an option to run PMD on diffs only? Like the one in https://github.com/marketplace/actions/pmd-analyser? Or do you plan to add such a helpful option?

@adangel
Copy link
Member

adangel commented Nov 29, 2021

No, not yet. I kept it simple for the beginning.
As this is an open source project, pull requests are always welcome.

Note: analyzing only the changed file might work with Apex, but maybe not for Java - the changed file might affect other files (which depend on the changed files), so to be correct, one would need to analyze the changed files + any file, that uses the changed file. This is not a problem yet for Apex, since we only look at the source code and don't resolve any types.

@adangel adangel added help wanted Extra attention is needed and removed help wanted Extra attention is needed labels Nov 29, 2021
@pmd pmd locked and limited conversation to collaborators Nov 29, 2021
@adangel adangel closed this as completed Nov 29, 2021
@adangel adangel added the question Further information is requested label Dec 1, 2021

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants