Skip to content

Add new quick-fixes#3756

Merged
datho7561 merged 1 commit intoeclipse-jdtls:mainfrom
jjohnstn:newquickfixes3
Apr 15, 2026
Merged

Add new quick-fixes#3756
datho7561 merged 1 commit intoeclipse-jdtls:mainfrom
jjohnstn:newquickfixes3

Conversation

@jjohnstn
Copy link
Copy Markdown
Contributor

  • add new methods to ModifierCorrectionSubProcessor and enable quick-fixes that call them in QuickFixProcessor
  • add new TypeArgumentMismatchSubProcessor class and enable quick-fix that uses it
  • add new tests to ModifierCorrectionsQuickFixTest and add new TypeArgumentMismatchCorrectionTest

@jjohnstn
Copy link
Copy Markdown
Contributor Author

Hi @datho7561 These are some more quick-fixes enabled by some upstream JDT refactoring work I did last week.

@datho7561 datho7561 moved this to Pending review in Java Tooling Apr 15, 2026
Copy link
Copy Markdown
Contributor

@datho7561 datho7561 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and works well. Thanks, Jeff!

@datho7561 datho7561 added this to the End of May 2026 milestone Apr 15, 2026
@datho7561
Copy link
Copy Markdown
Contributor

Hmm. There's conflicts that need to be resolved before I merge this

- add new methods to ModifierCorrectionSubProcessor and enable
  quick-fixes that call them in QuickFixProcessor
- add new TypeArgumentMismatchSubProcessor class and enable
  quick-fix that uses it
- add new tests to ModifierCorrectionsQuickFixTest and
  add new TypeArgumentMismatchCorrectionTest
@jjohnstn
Copy link
Copy Markdown
Contributor Author

Hi @datho7561 The rebase did a horrible job of merging the branch into main. I fixed it up and did a force push.

@datho7561 datho7561 merged commit e89afe4 into eclipse-jdtls:main Apr 15, 2026
2 of 3 checks passed
@github-project-automation github-project-automation Bot moved this from Pending review to Done in Java Tooling Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants