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

[WINDUPRULE-1015] Rules for APIs deprecated for removal #1012

Merged
merged 4 commits into from
Jul 28, 2023

Conversation

jmle
Copy link
Contributor

@jmle jmle commented Jul 26, 2023

@PhilipCattanach PhilipCattanach self-requested a review July 27, 2023 07:40
@PhilipCattanach
Copy link
Contributor

@jmle - I looked at the rules and there seems nothing to cover the following packages in the JDK18 sub-set
java.awt., java.lang., java.util., and javax.imageio.

@jmle
Copy link
Contributor Author

jmle commented Jul 27, 2023

@jmle - I looked at the rules and there seems nothing to cover the following packages in the JDK18 sub-set java.awt., java.lang., java.util., and javax.imageio.

Hi @PhilipCattanach, those methods are covered by this previously merged PR.

@PhilipCattanach
Copy link
Contributor

Thanks @jmle - Are we missing java.lang.Thread.stop ?
I can see all of the finalize and runFinalization methods are covered

@PhilipCattanach
Copy link
Contributor

@jmle
I'm getting an error
WindupRulesTest.testWindupRules:155
Error with test: deprecation-test-00005
Cause: (Rule: deprecation-test-00005) Assertion failed: deprecation-test-00005 - the hint was not found

However the rule has fired, once as expected and the message in the test looks right. I don't know why it is failing for me. Is it running locally OK for you?

@jmle
Copy link
Contributor Author

jmle commented Jul 27, 2023

@jmle I'm getting an error WindupRulesTest.testWindupRules:155 Error with test: deprecation-test-00005 Cause: (Rule: deprecation-test-00005) Assertion failed: deprecation-test-00005 - the hint was not found

However the rule has fired, once as expected and the message in the test looks right. I don't know why it is failing for me. Is it running locally OK for you?

@PhilipCattanach my bad, I was missing some escape dashes

@PhilipCattanach
Copy link
Contributor

@jmle The tests ran successfully.
I think the source technology tags in each of the rulesets needs to be updated
For example in deprecation-openjdk18.windup.xml
sourceTechnology id="openjdk" versionRange="(,18)"

which means <= 18
shouldn't that be <=17

@PhilipCattanach
Copy link
Contributor

D-oh! my mistake!
the version range is correct I was misreading it as (,18]
We are all good I can merge the PR

@PhilipCattanach PhilipCattanach merged commit 6ca369c into windup:master Jul 28, 2023
26 checks passed
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.

2 participants