AdaCore / ada_language_server

Server implementing the Microsoft Language Protocol for Ada and SPARK
GNU General Public License v3.0
228 stars 54 forks source link

[Feature Request] Disabling pop-up "Some project source directories are not listed..." #1006

Closed PaulCayard closed 2 years ago

PaulCayard commented 2 years ago

Every time I launch vscode with "Language Support for Ada" v23.0.8, the following pop-up appears: "Some project source directories are not listed in your workspace: do you want to add them?"

Is it possible to disable it?

Thanks a lot!

AnthonyLeonardoGracio commented 2 years ago

Hello @PaulCayard,

This popup indicates that some source directories of your project hierarchy are not listed under the current VS Code workspace: our extension then proposes to add them. Have you tried to click on 'Yes'? Once these directories get added to the workspace, the popup won't be shown again.

Regards,

PaulCayard commented 2 years ago

Hi Anthony, I don't want to add these directories to the workspace. If I click 'No', when I relaunch vscode pop-up is shown again. Thanks anyway for the explanation!

AnthonyLeonardoGracio commented 2 years ago

Ok I see. Then we can probably add a 'Do not show this again' option in that case. We thought that usually people want to have access to all the sources from the project.

reznikmm commented 2 years ago

An idea to discuss:

To propose adding directories only the user has an non-empty workspace.

Reason:

PaulCayard commented 2 years ago
  • In both cases he doesn't care about keeping workspace and the dialog is mostly useless to him.

This is my case. I use local git repositories as "vscode project" and I don't want to create a workspace.

(sorry for closing and reopening the thread, it was a mistake)

AnthonyLeonardoGracio commented 2 years ago

@PaulCayard Ok so @reznikmm solution is probably the way to go indeed (i.e: do nothing when there is no 'explicit' workspace).

AnthonyLeonardoGracio commented 2 years ago

@PaulCayard we have implemented the proposal, it will be available in the next release of the extension. Thanks for the suggestion!

PaulCayard commented 2 years ago

Yeah, I saw it in the edge branch. Thank you for the request implementation!

reznikmm commented 2 years ago

We have published 23.0.9 that includes these changes. It should be fixed.

PaulCayard commented 2 years ago

Thanks a lot! It works perfectly!