Skip to content
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

Can not save because the settings file is dirty #121804

Closed
isidorn opened this issue Apr 21, 2021 · 4 comments · Fixed by #169128
Closed

Can not save because the settings file is dirty #121804

isidorn opened this issue Apr 21, 2021 · 4 comments · Fixed by #169128
Assignees
Labels
config VS Code configuration, set up issues debt Code quality issues insiders-released Patch has been released in VS Code Insiders papercut 🩸 A particularly annoying issue impacting someone on the team verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@isidorn
Copy link
Contributor

isidorn commented Apr 21, 2021

Could we simply automatically save the file in this case and no need to show a notification?
Code pointer https://github.com/microsoft/vscode/blob/main/src/vs/workbench/services/workspaces/browser/abstractWorkspaceEditingService.ts#L300

That would make this a much smoother experience

Refs: #119463

@isidorn
Copy link
Contributor Author

isidorn commented Apr 21, 2021

Same use case here https://github.com/microsoft/vscode/blob/main/src/vs/workbench/services/configuration/common/configurationEditingService.ts#L259

Could we automatically save those configuration files so we do not get "file is dirty" errors

@sandy081 sandy081 added config VS Code configuration, set up issues debt Code quality issues labels Apr 22, 2021
@sandy081 sandy081 added this to the Backlog milestone Apr 22, 2021
@sandy081 sandy081 added verification-needed Verification of issue is requested papercut 🩸 A particularly annoying issue impacting someone on the team labels Dec 14, 2022
sandy081 added a commit that referenced this issue Dec 14, 2022
sandy081 added a commit that referenced this issue Dec 14, 2022
@VSCodeTriageBot VSCodeTriageBot added the unreleased Patch has not yet been released in VS Code Insiders label Dec 14, 2022
@sandy081 sandy081 modified the milestones: Backlog, January 2023 Dec 14, 2022
weartist pushed a commit to weartist/vscode that referenced this issue Dec 15, 2022
@VSCodeTriageBot VSCodeTriageBot added insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Dec 15, 2022
@connor4312 connor4312 added the verified Verification succeeded label Jan 25, 2023
@connor4312
Copy link
Member

Are there steps for this? I tried making a conflicting dirty settings files, but I don't think this is what you were talking about...

image

@connor4312 connor4312 added verification-steps-needed Steps to verify are needed for verification and removed verified Verification succeeded labels Jan 25, 2023
@sandy081
Copy link
Member

To verify

  • Disable autosave
  • Open settings.json file and make it dirty
  • Open settings UI and change some settings
  • Make sure setting is applied and file is saved.

@sandy081 sandy081 removed the verification-steps-needed Steps to verify are needed for verification label Jan 26, 2023
@karthiknadig karthiknadig added the verified Verification succeeded label Jan 26, 2023
@karthiknadig
Copy link
Member

Thanks for the steps @sandy081 that worked.

@github-actions github-actions bot locked and limited conversation to collaborators Feb 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
config VS Code configuration, set up issues debt Code quality issues insiders-released Patch has been released in VS Code Insiders papercut 🩸 A particularly annoying issue impacting someone on the team verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants