Skip to content

Improve ResolveMainClassHandler.resolveMainClassUnderPaths#596

Merged
wenytang-ms merged 2 commits into
microsoft:mainfrom
snjeza:issue-595
Aug 13, 2025
Merged

Improve ResolveMainClassHandler.resolveMainClassUnderPaths#596
wenytang-ms merged 2 commits into
microsoft:mainfrom
snjeza:issue-595

Conversation

@snjeza

@snjeza snjeza commented Jul 20, 2025

Copy link
Copy Markdown
Contributor

Fixes #595

@wenytang-ms wenytang-ms requested a review from Copilot August 6, 2025 06:51

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves the ResolveMainClassHandler.resolveMainClassUnderPaths method to provide more targeted main class resolution by filtering projects based on provided paths instead of searching all Java projects.

  • Adds conditional logic to filter Java projects when specific paths are provided
  • Removes REFERENCED_PROJECTS flag from search scope to focus only on source files
  • Imports JavaCore utility for creating IJavaProject instances

@snjeza

snjeza commented Aug 6, 2025

Copy link
Copy Markdown
Contributor Author

I have updated the PR.

@wenytang-ms wenytang-ms requested a review from chagong as a code owner August 13, 2025 07:12
@wenytang-ms wenytang-ms merged commit a209e39 into microsoft:main Aug 13, 2025
4 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.

Improve ResolveMainClassHandler.resolveMainClassUnderPaths

3 participants