A RetroSearch Logo

Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Search Query:

Showing content from https://github.com/pmd/pmd/issues/2435 below:

[test] Remove duplicated Dummy language module · Issue #2435 · pmd/pmd · GitHub

Is your feature request related to a problem? Please describe.
There's currently a "dummy" language module in the pmd-core test sources and one in the main pmd-test sources. They're a pain to keep in sync on the 7.0.x branch, and they're perfect duplicates. It also seems like the only usages of those DummyNodes in the pmd-test module are... in the tests of pmd-test

Describe the solution you'd like
Just make the test sources of pmd-test depend on the test sources of pmd-core, and remove the dummy module from pmd-test. https://stackoverflow.com/questions/174560/sharing-test-code-in-maven

This would remove the ability to use dummy nodes in external code that depends on pmd-test, but I think, they may use a real language module to fake nodes? I can't really think of a reason why anyone would want to build fake trees by hand outside of our test sources

Describe alternatives you've considered

  1. Don't change anything, keep syncing the sources manually. This is a pain
  2. Put the "dummy" language module in the main sources of PMD core. This would have some advantages, but then we'd have to make a commitment to binary compatibility over versions... and it would show up in LanguageRegistry

RetroSearch is an open source project built by @garambo | Open a GitHub Issue

Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo

HTML: 3.2 | Encoding: UTF-8 | Version: 0.7.4