Skip to content

[core] Deprecate utility classes#2060

Merged
adangel merged 10 commits into
pmd:masterfrom
oowekyala:deprecate-utils
Oct 14, 2019
Merged

[core] Deprecate utility classes#2060
adangel merged 10 commits into
pmd:masterfrom
oowekyala:deprecate-utils

Conversation

@oowekyala

Copy link
Copy Markdown
Member

Deprecate contents of nspmd.util as they're internal API. Many of these can be completely removed with 7.0.0, eg Filters, EmptyIterator, FileIterable, etc. Sometimes there's already an alternative in JDK 7.

@oowekyala oowekyala added the is:deprecation The main focus is deprecating public APIs or rules, eg to make them internal, or removing them label Oct 9, 2019
@oowekyala oowekyala added this to the 6.19.0 milestone Oct 9, 2019
Comment thread pmd-core/src/main/java/net/sourceforge/pmd/util/FileUtil.java Outdated
Comment thread pmd-core/src/main/java/net/sourceforge/pmd/Report.java Outdated
Comment thread pmd-core/src/main/java/net/sourceforge/pmd/Report.java Outdated
oowekyala and others added 3 commits October 10, 2019 17:27
Co-Authored-By: Juan Martín Sotuyo Dodero <juansotuyo@gmail.com>
@ghost

ghost commented Oct 10, 2019

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 pmd-core/src/main/java/net/sourceforge/pmd/cpd/GridBagHelper.java
Comment thread pmd-core/src/main/java/net/sourceforge/pmd/util/ClassUtil.java
@adangel adangel merged commit ecf9be0 into pmd:master Oct 14, 2019
@oowekyala oowekyala deleted the deprecate-utils branch April 20, 2020 02:06
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.

3 participants