Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

Build complete workspace when buildOnSave is set to workspace #1170

Closed
sandy081 opened this issue Aug 29, 2017 · 1 comment
Closed

Build complete workspace when buildOnSave is set to workspace #1170

sandy081 opened this issue Aug 29, 2017 · 1 comment
Assignees

Comments

@sandy081
Copy link
Member

Testing microsoft/vscode#33297

When the setting go.buildOnSave is set to workspace, in a multi-root workspace it is expected to build the complete workspace instead of just the workspace folder. I see it was mentioned expected but this can be changed.

Here is the use case: In the sample MR set up I have two root folders hello and stringutil. I wanted to configure settings in stringutil folder that on changes in this folder, I would like to build all other workspace folders too.

@ramya-rao-a
Copy link
Contributor

Closing this as "as-designed"
For now, in multi-root set-up, in the Go extension workspace in build/lint/vet on save settings will not mean the entire workspace. Build/Lint/Vetting the entire workspace will get expensive and there are no great benefits for the same.

We will re-visit if and when there is user feedback in this direction

@vscodebot vscodebot bot locked and limited conversation to collaborators Jul 16, 2018
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants