vscode-language-renpy icon indicating copy to clipboard operation
vscode-language-renpy copied to clipboard

[Bug] - `files.exclude` is always updated even when renpy.excludeCompiledFilesFromWorkspace is false

Open multimokia opened this issue 1 year ago • 6 comments

Would it be possible to make it so we can opt out of the automatic generation of files.exclude in local settings?

If I keep "renpy.excludeCompiledFilesFromWorkspace": false, in my main settings.json file, I notice every time I load a project, there's untracked changes to my local settings.json as the extension keeps adding the compiled files to it, even if the patterns are all set to off.

However this behaviour seems redundant as there's no need to add them there. Personally I'd love it if this wouldn't keep being re-added!

multimokia avatar Aug 31 '22 19:08 multimokia

I was hoping the lasted merge from @LuqueDaniel fixed this. (See https://github.com/LuqueDaniel/vscode-language-renpy/issues/172) Are you on the new version? (Released just now)

duckdoom4 avatar Aug 31 '22 22:08 duckdoom4

The conditional will have to be modified to check the value of renpy.excludeCompiledFilesFromWorkspace.

https://github.com/LuqueDaniel/vscode-language-renpy/blob/7fab9218640384ed8fb473045d67fe8f5dd8aaa0/src/extension.ts#L77-L81

LuqueDaniel avatar Aug 31 '22 23:08 LuqueDaniel

Can confirm this is still on latest

multimokia avatar Sep 01 '22 04:09 multimokia

Yes, because what I fixed was the overwriting of other values that may have files.exclude #172 .

I'll change it so that it only writes the values in case the renpy.excludeCompiledFilesFromWorkspace option is enabled.

LuqueDaniel avatar Sep 01 '22 11:09 LuqueDaniel

Awesome, thank you!

multimokia avatar Sep 01 '22 15:09 multimokia

@LuqueDaniel We should compare the values in the config to the new values. Cus we need to make sure that if 'hide' is set to false, it also updates the config values.

As mentioned in #172, I've also done this with the other config here. (But feel free to do it in another way.)

duckdoom4 avatar Sep 01 '22 19:09 duckdoom4