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

follow the deprecation advice and use the replacement classes to get rid of some deprecation warnings in the build log #1540

Merged

Conversation

carstenartur
Copy link
Contributor

Currently we have more than 400 warnings in the jdt.ui build log.

What it does

This change reduces the amount of deprecations of a certain kind that is used in the cleanups.

How to test

Run the cleanups, where the change is invalid it should cause issues.

Author checklist

@carstenartur carstenartur changed the title 431 warnings in the log is enough, we should reduce the amount follow the deprecation advise and use the replacement classes to get rid of some deprecation warnings in the build log Jul 25, 2024
@carstenartur carstenartur marked this pull request as ready for review July 25, 2024 12:14
@carstenartur carstenartur changed the title follow the deprecation advise and use the replacement classes to get rid of some deprecation warnings in the build log follow the deprecation advice and use the replacement classes to get rid of some deprecation warnings in the build log Jul 26, 2024
@carstenartur carstenartur force-pushed the ReduceAmountOfDeprecationWarnings branch 2 times, most recently from 723ef06 to 6aed53e Compare August 1, 2024 13:23
@carstenartur carstenartur force-pushed the ReduceAmountOfDeprecationWarnings branch from 6aed53e to 378e170 Compare August 3, 2024 09:58
@carstenartur carstenartur force-pushed the ReduceAmountOfDeprecationWarnings branch from 378e170 to a19b17a Compare September 4, 2024 16:14
@carstenartur carstenartur force-pushed the ReduceAmountOfDeprecationWarnings branch from a19b17a to 0fa4b0c Compare September 5, 2024 03:08
@carstenartur
Copy link
Contributor Author

image

@jjohnstn jjohnstn merged commit 1df6c00 into eclipse-jdt:master Sep 5, 2024
9 checks passed
@jjohnstn
Copy link
Contributor

jjohnstn commented Sep 5, 2024

Thanks @carstenartur

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