Skip to content

Fix deprecated API usage#4777

Merged
adangel merged 3 commits into
pmd:masterfrom
adangel:fix-deprecated-api-usage
Jan 26, 2024
Merged

Fix deprecated API usage#4777
adangel merged 3 commits into
pmd:masterfrom
adangel:fix-deprecated-api-usage

Conversation

@adangel

@adangel adangel commented Dec 16, 2023

Copy link
Copy Markdown
Member

Describe the PR

  • Fixes deprecated API usage so that we can delete this on core.
  • Fixes in pmd-doc, html, javascript, jsp, modelica, objectivec, scala, swift, vm
  • Also fixes a small bug in core/BaseAntlrNode::getTextRegion (which I used in swift).

Related issues

  • None

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)

Fixes in pmd-doc, html, javascript,
jsp, modelica, objectivec, scala,
swift, vm
@adangel adangel added the in:pmd-internals Affects PMD's internals label Dec 16, 2023
@adangel adangel added this to the 7.0.0 milestone Dec 16, 2023
@ghost

ghost commented Dec 16, 2023

Copy link
Copy Markdown
1 Message
📖 Compared to master:
This changeset changes 0 violations,
introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 0 errors and 0 configuration errors.
Download full report as build artifact
Compared to master:
This changeset changes 0 violations,
introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 0 errors and 0 configuration errors.
Download full report as build artifact

Generated by 🚫 Danger

@adangel adangel mentioned this pull request Dec 19, 2023
4 tasks
@adangel adangel merged commit 88239b7 into pmd:master Jan 26, 2024
@adangel adangel deleted the fix-deprecated-api-usage branch January 26, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in:pmd-internals Affects PMD's internals

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant