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

fix Unnecessary @SuppressWarnings("deprecation") #706

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jukzi
Copy link
Contributor

@jukzi jukzi commented Nov 26, 2024

jdt does not require @SuppressWarnings("deprecation")
for imports anymore.

Copy link

github-actions bot commented Nov 26, 2024

Test Results

  663 files  ±0    663 suites  ±0   1h 15m 44s ⏱️ + 2m 10s
2 211 tests ±0  2 164 ✅ ±0   47 💤 ±0  0 ❌ ±0 
6 777 runs  ±0  6 634 ✅ ±0  143 💤 ±0  0 ❌ ±0 

Results for commit 2bd7605. ± Comparison against base commit 5810f68.

♻️ This comment has been updated with latest results.

@jukzi
Copy link
Contributor Author

jukzi commented Dec 2, 2024

@HannesWell this repo does not have your version increment bot yet?
20:20:17 [ERROR] Failed to execute goal org.eclipse.tycho.extras:tycho-p2-extras-plugin:4.0.10:compare-version-with-baselines (compare-attached-artifacts-with-release) on project org.eclipse.equinox.common.tests: Only qualifier changed for (org.eclipse.equinox.common.tests/3.17.200.v20241128-1855). Expected to have bigger x.y.z than what is available in baseline (3.17.200.v20240513-1104) -> [Help 1]

@HannesWell
Copy link
Member

@HannesWell this repo does not have your version increment bot yet?

Yes, that's unfortunately right, see #678.
I didn't have the time for a 'perfect' solution in Tycho yet and didn't want to spend extra time for a work-around just for Equinox. But maybe I should do the latter.

jdt does not require @SuppressWarnings("deprecation")
 for imports anymore.
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.

3 participants