Skip to content

[core] Deprecate support for List attributes with XPath 2.0#2609

Merged
adangel merged 3 commits into
pmd:masterfrom
adangel:deprecate-xpath-lists
Jun 26, 2020
Merged

[core] Deprecate support for List attributes with XPath 2.0#2609
adangel merged 3 commits into
pmd:masterfrom
adangel:deprecate-xpath-lists

Conversation

@adangel

@adangel adangel commented Jun 20, 2020

Copy link
Copy Markdown
Member

Describe the PR

Deprecates any access to List attributes with XPath queries.
I decided against marking the 3 getters in apex with @DeprecatedAttribute, since that would not cover any new getter that might be created in the future.

I don't think, we need to put this more prominently in the release notes, since this feature is probably not used very often (if at all).

Related issues

Ready?

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

@adangel adangel added the is:deprecation The main focus is deprecating public APIs or rules, eg to make them internal, or removing them label Jun 20, 2020
@adangel adangel added this to the 6.25.0 milestone Jun 20, 2020
@ghost

ghost commented Jun 20, 2020

Copy link
Copy Markdown
1 Message
📖 This changeset introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 0 errors and 0 configuration errors.
Full report
This changeset introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 0 errors and 0 configuration errors.
Full report

Generated by 🚫 Danger

Comment thread docs/pages/pmd/userdocs/extending/writing_xpath_rules.md Outdated
Comment thread pmd-core/src/main/java/net/sourceforge/pmd/lang/ast/xpath/Attribute.java Outdated
@adangel adangel self-assigned this Jun 26, 2020
@adangel adangel merged commit 842bdb0 into pmd:master Jun 26, 2020
@adangel adangel deleted the deprecate-xpath-lists branch June 26, 2020 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

is:deprecation The main focus is deprecating public APIs or rules, eg to make them internal, or removing them

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[core] Deprecate support for List attributes with XPath 2.0

2 participants