[core] Support configurable tabsize for AST node position in reports
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 5.5k
- Forks
- 1.6k
- Avg merge
- 1d 19h
- Merged PRs (30d)
- 54
Description
See #288 for an example.
There should be a property, that is configured via commandline/ant/... to configure the tabsize.
It's currently at the default value of 8.
Note: At least the Designer needs to be adjusted, too - since it also has a special tab handling, which is currently 8.
If the tabsize doesn't match your editor configuration, then you might have trouble finding the reported violation position.
And consider setting the tabsize to 4 by default for all languages, since this is the most common setting nowadays.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with issue #288 and the tab-position reporting path, then inspect pmd-core/src/main/java/net/sourceforge/pmd/util/designer/CodeEditorTextPane.java, especially its special tab handling. Trace how configuration is exposed through the command line and Ant, and consider the requested default; done means reported positions and the Designer use the configured tab size.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- cli, tooling
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100