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

Hot Exit: Restore files after an editor crashes #12327

Closed
BirdLeeSCUT opened this issue Sep 20, 2016 · 10 comments
Closed

Hot Exit: Restore files after an editor crashes #12327

BirdLeeSCUT opened this issue Sep 20, 2016 · 10 comments
Assignees
Labels
feature-request Request for new features or functionality workbench-hot-exit Preservation of unsaved changes across restarts
Milestone

Comments

@BirdLeeSCUT
Copy link

BirdLeeSCUT commented Sep 20, 2016

Vscode is so excellent , we love it very much! But this days vscode still has some bugs making it crash, especially when I work with big project that open so many files at the same time. And when vscode reopenes after crashed, the file openning status rolls back to that when I openned it last time, not the status when it alert "no responding". This make some inconvenience for us to contunue our work. So I hope vscode can save the file openning status when it crash.

@BirdLeeSCUT BirdLeeSCUT changed the title Feature request: save the file openning status when it crash Feature request: save the file openning status when vscode crashes Sep 20, 2016
@chrmarti chrmarti added the feature-request Request for new features or functionality label Sep 20, 2016
@chrmarti
Copy link
Collaborator

Related to #101

@Tyriar Tyriar closed this as completed Sep 20, 2016
@Tyriar Tyriar added *duplicate Issue identified as a duplicate of another issue(s) and removed *duplicate Issue identified as a duplicate of another issue(s) labels Sep 20, 2016
@Tyriar
Copy link
Member

Tyriar commented Sep 21, 2016

I'm going to track this feature separately to #101

@Tyriar Tyriar changed the title Feature request: save the file openning status when vscode crashes Feature request: Restore files after an editor crashes Sep 21, 2016
@Tyriar Tyriar reopened this Sep 21, 2016
@Tyriar Tyriar changed the title Feature request: Restore files after an editor crashes Hot Exit: Restore files after an editor crashes Oct 14, 2016
@Tyriar Tyriar added the file-explorer Explorer widget issues label Oct 14, 2016
@Tyriar Tyriar added workbench-hot-exit Preservation of unsaved changes across restarts workbench and removed file-explorer Explorer widget issues labels Oct 14, 2016
@Tyriar Tyriar modified the milestone: Backlog Nov 1, 2016
@Tyriar Tyriar removed the workbench label Nov 4, 2016
@Tyriar
Copy link
Member

Tyriar commented Nov 11, 2016

I believe this is mostly working after progress in #101, it will currently not attempt to restore the changes until VS Code has been completely exited though. We probably want a dialog asking whether the user wants to restore.

@bpasero
Copy link
Member

bpasero commented Nov 18, 2016

Needs #15718

@bpasero
Copy link
Member

bpasero commented Nov 29, 2016

@Tyriar should we look into enabling this scenario now that I landed 6422acb ?

Imho the only thing needed is to remove the checks for hot exit being enabled in backupModelTracker.

@Tyriar
Copy link
Member

Tyriar commented Nov 29, 2016

The checks must remain for the extension development case. I suggest we support crash recovery on empty workspaces when we support hot exit on empty workspaces (which should be soon). #13733

@bpasero
Copy link
Member

bpasero commented Nov 30, 2016

@Tyriar sure for extension development we can have another check to not do backups. I am not sure how crash recovery and empty workspaces are related, I could have a folder opened with dirty files and a crash takes down my changes. I would want them to restore even if hot exit is disabled.

@Tyriar
Copy link
Member

Tyriar commented Dec 3, 2016

Backups should be happening all the time now due to empty workspace restore 🎆

@Tyriar Tyriar closed this as completed Dec 3, 2016
@bpasero bpasero added this to the November 2016 milestone Dec 3, 2016
@bpasero bpasero removed this from the Backlog milestone Dec 3, 2016
@bpasero
Copy link
Member

bpasero commented Dec 3, 2016

@Tyriar does not work for me:

  • make some files dirty
  • kill the process
  • restart
    => no restore

I think you still check for isHotExitEnabled in backupModelTracker.ts?

@bpasero bpasero reopened this Dec 3, 2016
@Tyriar Tyriar closed this as completed in 8c97220 Dec 4, 2016
@Tyriar
Copy link
Member

Tyriar commented Dec 5, 2016

Test plan #16544

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality workbench-hot-exit Preservation of unsaved changes across restarts
Projects
None yet
Development

No branches or pull requests

4 participants