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.

Sign up for GitHub

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

Jump to bottom

feat(linters): add pmd #717

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

feat(linters): add pmd #717

wants to merge 15 commits into from

Conversation

kmoschcau
Copy link
Contributor

This adds the PMD linter.

kmoschcau added a commit to kmoschcau/nvim-config that referenced this pull request Jan 4, 2025
@kmoschcau
feat(nvim-lint): add PMD and thus remove none-ls
5da733c 
Relates to:
- mfussenegger/nvim-lint#717
- mfussenegger/nvim-lint#718
@mfussenegger
Copy link
Owner

Given that pmd supports sarif I guess this can be updated after #718 is in?

@kmoschcau
Copy link
Contributor Author

Oh yeah sure! I didn't even realize they added this since I last used it.

@kmoschcau
Copy link
Contributor Author

OK I made quite a few more changes to the SARIF parser for PMD and added more tests. @mfussenegger should I backport all of that to the checkstyle branch, or is it fine with you to merge that with the PMD branch?

@kmoschcau
Copy link
Contributor Author

@mfussenegger The checkstyle code base has been a pain to work with so far. I don't think there is much going to happen there soon. Would you merge this PR with the SARIF-Parser and PMD integration, if I revert the changes to checkstyle?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kmoschcau @mfussenegger