Skip to content

[java] Fix #6308: Add syntax highlighting to MarkdownRenderer#6605

Merged
adangel merged 6 commits into
pmd:mainfrom
UncleOwen:issue-6308-CPD-MarkdownRenderer-Add-syntax-highlighting
May 8, 2026
Merged

[java] Fix #6308: Add syntax highlighting to MarkdownRenderer#6605
adangel merged 6 commits into
pmd:mainfrom
UncleOwen:issue-6308-CPD-MarkdownRenderer-Add-syntax-highlighting

Conversation

@UncleOwen
Copy link
Copy Markdown
Member

Describe the PR

Add syntax highlighting to MarkdownRenderer

Related issues

Ready?

  • Added unit tests for fixed bug/feature
  • Passing all unit tests
  • Complete build ./mvnw clean verify passes (checked automatically by github actions)
  • Added (in-code) documentation (if needed)

@codacy-production
Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 1 medium

Alerts:
⚠ 1 issue (≤ 0 issues of at least minor severity)

Results:
1 new issue

Category Results
BestPractice 1 medium

View in Codacy

🟢 Metrics 0 complexity · 2 duplication

Metric Results
Complexity 0
Duplication 2

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@pmd-actions-helper
Copy link
Copy Markdown
Contributor

pmd-actions-helper Bot commented Apr 19, 2026

Documentation Preview

No relevant source code has been changed, pmdtester skipped.

(comment created at 2026-05-08 07:58:26+00:00 for 16c3d3b)

@UncleOwen
Copy link
Copy Markdown
Member Author

The one Codacy issue is in fact a FP in PMD: #6606

Convert the remaining uses of makeReport to CpdReportBuilder.
@UncleOwen UncleOwen force-pushed the issue-6308-CPD-MarkdownRenderer-Add-syntax-highlighting branch from 068c079 to e7eed37 Compare April 24, 2026 14:14
@adangel adangel added this to the 7.25.0 milestone May 8, 2026
Copy link
Copy Markdown
Member

@adangel adangel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Comment thread pmd-core/src/main/java/net/sourceforge/pmd/cpd/CPDReport.java
Comment thread pmd-core/src/test/java/net/sourceforge/pmd/cpd/CpdTestUtils.java Outdated
Comment thread pmd-core/src/test/java/net/sourceforge/pmd/cpd/CpdTestUtils.java Outdated
@adangel adangel merged commit fe59c70 into pmd:main May 8, 2026
2 checks passed
@UncleOwen UncleOwen deleted the issue-6308-CPD-MarkdownRenderer-Add-syntax-highlighting branch May 8, 2026 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[core] CPD Markdown format: Add syntax highlighting

2 participants