Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate from com.google.common.base.Predicate to java.util.function.Predicate #208

Merged
merged 1 commit into from
Mar 2, 2021

Conversation

basil
Copy link
Member

@basil basil commented Jan 31, 2021

Despite the fact that com.google.common.base.Predicate is used in public APIs, it is safe to migrate this to java.util.function.Predicate.

Searching for API usages in sources

The only references in sources are to a copy of Stapler in the io.jenkins.blueocean.commons.stapler.export package namespace, apparently added in jenkinsci/blueocean-plugin#711. These references are to a different package and we are not affecting them in this change.

Searching for API usages in binaries

Creating /tmp/additionalClasses with

org/kohsuke/stapler/export/FilteringTreePruner

and /tmp/additionalFields with

org.kohsuke.stapler.export.Model#HAS_PROPERTY_NAME
org.kohsuke.stapler.export.Model#HAS_PROPERTY_NAME_IN_ANCESTRY

then using jenkins-infra/usage-in-plugins to look for usages in plugins, including those in CloudBees CI, with

mvn process-classes exec:exec -Dexec.executable=java -Dexec.args='-classpath %classpath org.jenkinsci.deprecatedusage.Main --additionalClasses /tmp/additionalClasses --additionalFields /tmp/additionalFields --onlyIncludeSpecified --updateCenter https://jenkins-updates.cloudbees.com/update-center/envelope-core-oc/update-center.json?version=2.263.2.3,https://jenkins-updates.cloudbees.com/update-center/envelope-core-mm/update-center.json?version=2.263.2.3'

produced no references.

@jglick jglick added the chore label Mar 2, 2021
@jglick jglick merged commit 1f497fb into jenkinsci:master Mar 2, 2021
@basil basil deleted the predicate branch March 2, 2021 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants