-
Notifications
You must be signed in to change notification settings - Fork 102
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
vsCode settings are not updated #432
Comments
@maybeec thanks for reporting the issue.
|
Hi @hohwille, we configured I am not sure whether this progam is running. At least I cannot see anything on the logs:
|
@maybeec thanks for sharing all these additional details. 👍
Can you check your disc if that file exists and is writable?
FYI: I am not a user of VS Code as my main IDE is Eclipse. We would need more invonvement here from users of other IDEs to get IntelliJ and VS Code properly supported... |
From the shell-script code I checked/verified that whenever VS Code is launched via devon command in advance the config sync will take place:
So maybe the expectation that |
The log issue might be related to microsoft/vscode#64897 |
Reading the logs once more, it seems that the installation is done asynchronously which causes multiple installation processes being spawned in parallel it seems. Maybe we'll find a way to do it synchronously. |
Even when
is called, the
therein is only called, when
is true. Means, it sync from settings directory when directory If the directory already exists, it needs an explicit `call of
to update the settings. At least that is, what I see in the script code. And this need for an explicit I personally also would have expected, the initial sync from settings is done during a call of |
No I think settings should be updated as well on Devon ide update all. This would be consistent to what eclipse settings are doing and it is even more important if you want to roll out some new settings to your team. Regarding the plugin installation I found a fix which I will soon provide as a pr. |
I think
is the problem. There are global settings and project specific settings. Project specific settings can be easily committed with the project into git in the .vscode folder right in the source code. It seems, that this line of code is addressing this .vscode folder, which does not make sense to me. Instead, the folder on conf/.vscode is the critical folder to be updated. Maybe the intention of not updating conf/.vscode is related to the fact, that also conf/.m2/settings.xml cannot be updated as well. Anyhow, for vsCode this would be a more frequently changed file, which we should try to keep up to date by scripts. |
I can try to create a PR to fix the rest of this issue. IMHO this should be analogue to eclipse so when vscode is launched, the settings are updated. |
Current observations are not entirely correct about this:
If the workspace (
So the actual error is just, that
Like it is done for eclipse:
So trivial fix, but not so easy to trace down... |
I created a PR with the according fix. However, now I am facing the next error with the expected config provided in the steps to reproduce:
I have to dig what is causing this error. Actually boolean literals like |
I extended the JUnit tests with this |
OK, analysis completed: I was chasing a legacy error as for my testing I am using symlinks to the most recent devonfw-ide sources. As this is not working for the JARs in the lib folder that get deleted by maven and have chaning version numbers, I manually copy |
Expected behavior
As a developer, I want to share vsCode settings so that everyone in my team works for example with the same linting settings.
Actual behavior
I updated the settings in
settings/vscode/workspace/update/.vscode/settings.json
and randevon ide update all
, but the settings are not updated. Even on a fresh installation, the settings are not propagated to the user settings.Steps to reproduce (bug) / Use Case of feature request (enhancement)
settings/vscode/workspace/update/.vscode/settings.json
of your current IDE installation with the content{ "tslint.autoFixOnSave": true }
.devon ide update all
conf/vscode/User/settings.json
will not contain the new or update value.Related/Dependent Issues
Comments/Hints:
Affected version:
The text was updated successfully, but these errors were encountered: