-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
[vm] Deprecate language module? #2176
Copy link
Copy link
Closed
Labels
a:questionRequest for information that doesn't necessarily entail changes to the code baseRequest for information that doesn't necessarily entail changes to the code base
Metadata
Metadata
Assignees
Labels
a:questionRequest for information that doesn't necessarily entail changes to the code baseRequest for information that doesn't necessarily entail changes to the code base
Type
Fields
Give feedbackNo fields configured for issues without a type.
I'm not sure whether the VM language module is worth the effort of keeping it up to date with PMD 7.x. Nobody mentioned it on github/sourceforge/stackoverflow since 2014 ---I think this is the most recent mention of it, and there's nothing about it on either sourceforge or github apart from us keeping it up to date.
Besides, the codebase looks copy-pasted from the original velocity repo, with many unused things that only made sense in that codebase. It duplicates Javacc-generated code, duplicates functionality of AbstractNode, etc. I think it will get worse and worse unless we rehaul it somehow. But without a community showing interest in it, is this really worthwhile? I would argue for deprecating it unless someone takes it over