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

Un-deprecate ListSelectionDialog constructor #2733

Merged
merged 1 commit into from
Jan 19, 2025

Conversation

akurtakov
Copy link
Member

Deprecation has been done when Builder was added which seems to be a trend in the Java world.
Undeprecation is done now as :

  • it's one of the very few (if there are any others) APIs that "mandate" going through a builder, which makes it inconsistent with rest
  • documentation hasn't been adjusted accordingly (even the class level javadoc has example using the deprecated constructor)
  • usages in the SDK hasn't been migrated to the builder approach (for years) thus
    generating a number of deprecation warnings

Deprecation could be added again whenever the previous issues are resolved.

Deprecation has been done when Builder was added which seems to be a
trend in the Java world.
Undeprecation is done now as :
* it's one of the very few (if there are any others) APIs that "mandate"
going through a builder, which makes it inconsistent with rest
* documentation hasn't been adjusted accordingly (even the class level
javadoc has example using the deprecated constructor)
* usages in the SDK hasn't been migrated to the builder approach (for
years) thus
generating a number of deprecation warnings

Deprecation could be added again whenever the previous issues are
resolved.
Copy link
Contributor

Test Results

 1 818 files  ±0   1 818 suites  ±0   1h 28m 38s ⏱️ - 1m 36s
 7 739 tests ±0   7 510 ✅  - 1  228 💤 ±0  1 ❌ +1 
24 378 runs  ±0  23 628 ✅  - 1  749 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit da92c3c. ± Comparison against base commit d6af2f2.

@akurtakov
Copy link
Member Author

Failing test unrelated to un-deprecation of API.

@akurtakov akurtakov merged commit 75f86b9 into eclipse-platform:master Jan 19, 2025
15 of 17 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.

1 participant