Skip to content

Only report build failure when the build errors are on the running project's classpath#459

Merged
testforstephen merged 1 commit into
microsoft:mainfrom
testforstephen:jinbo_build
Nov 21, 2022
Merged

Only report build failure when the build errors are on the running project's classpath#459
testforstephen merged 1 commit into
microsoft:mainfrom
testforstephen:jinbo_build

Conversation

@testforstephen
Copy link
Copy Markdown
Contributor

  • For a build-tool managed project, will check if the running project and its references projects contain build errors. If so, report build failure. Otherwise, return succeed.
  • For unmanaged folder, only check if the running main class contains build errors. If so, report build failure. Otherwise, return succeed.

@jdneo
Copy link
Copy Markdown
Member

jdneo commented Nov 17, 2022

I'm thinking that how we can re-use the code in https://github.com/eclipse/eclipse.jdt.ls/blob/d4101ae14289ff7db98763cf731a0c74138589b8/org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/handlers/BuildWorkspaceHandler.java#L100

Now we build the entire workspace but only check the markers for the trigger project. That means we may waste some time for building unrelated projects.

@testforstephen
Copy link
Copy Markdown
Contributor Author

I'm thinking that how we can re-use the code in https://github.com/eclipse/eclipse.jdt.ls/blob/d4101ae14289ff7db98763cf731a0c74138589b8/org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/handlers/BuildWorkspaceHandler.java#L100

Now we build the entire workspace but only check the markers for the trigger project. That means we may waste some time for building unrelated projects.

I will hold on current behavior first before the performance becomes a concern.

@testforstephen testforstephen merged commit 4384d85 into microsoft:main Nov 21, 2022
@testforstephen testforstephen deleted the jinbo_build branch November 21, 2022 02:08
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