▲ 865 ▼ Microsoft Please Fix (lemmy.zip) submitted 2 years ago* (last edited 2 years ago) by Maven@lemmy.zip to c/programmer_humor@programming.dev 331 comments fedilink hide all child comments The real report
[+] SARGE@startrek.website 14 points 2 years ago* (last edited 1 week ago) (1 child) [deleted] permalink fedilink source parent hideshow 2 child comments replies: [–] bamboo@lemmy.blahaj.zone 29 points 2 years ago (1 child) reading through it, it sounds like they opened a project in VSCode, and it saw that there was a local git repo already initialized, with 3 months of changes uncommitted and not staged. So the options there are to stage the changes (git add) to be committed or discard the changes (git checkout -- .). I guess they chose the discard option thinking it was a notification and i guess the filename would be added to gitignore or something? Instead, it discarded the changes, and to the user, it looked like VSCode did rm -rf and not that this was the behavior of git. Since the changes were never committed, even git reflog can't save them. permalink fedilink source parent hideshow 2 child comments replies: [–] Mad_Punda@feddit.org 30 points 2 years ago (3 children) From this issue: https://github.com/microsoft/vscode/issues/32459 It appears that the behavior actually included a git clean. Which is insane in my opinion. Not sure if they changed it since, but there’s definitely a dev defending it. permalink fedilink source parent hideshow 6 child comments replies: [–] MajorHavoc@programming.dev 12 points 2 years ago It appears that the behavior actually included a git clean. Which is insane in my opinion. Yeah. Building a convenient accessible context free way to run git clean...sure feels like the actions of someone who just wants to watch the world burn. permalink fedilink source parent [–] bamboo@lemmy.blahaj.zone 3 points 2 years ago (1 child) Yeah, it's unclear to me at the time if the dialogue box in the screenshot appeared when doing a select all operation, but it reads as though the OP dev didn't understand git, discarded their work, and got upset that it was an option. Realistically if the dialogue box appeared, I'm not sure there would be anything else the IDE could do to prevent the dev from themselves. Perhaps reject operations affecting 5000 files? But then you'll just have someone with the same issue for 4000 files. permalink fedilink source parent hideshow 2 child comments replies: [–] Mad_Punda@feddit.org 13 points 2 years ago* The issue I linked has a very good analysis of the UX issues and several suggestions for fixing these. They went with a minor iteration on the original message box, which not only includes a clearer message and the number of files affected, but also defaults to not touching untracked files (while preserving the option to delete untracked files as before). permalink fedilink source parent [–] Scoopta@programming.dev 2 points 2 years ago (1 child) He said they're not going to change it, just make the dialog a lot more clear and add a second button to it that will only do a reset without the clean. permalink fedilink source parent hideshow 2 child comments replies: [–] BatmanAoD@programming.dev 6 points 2 years ago The second button is actually a pretty major change! permalink fedilink source parent
[–] bamboo@lemmy.blahaj.zone 29 points 2 years ago (1 child) reading through it, it sounds like they opened a project in VSCode, and it saw that there was a local git repo already initialized, with 3 months of changes uncommitted and not staged. So the options there are to stage the changes (git add) to be committed or discard the changes (git checkout -- .). I guess they chose the discard option thinking it was a notification and i guess the filename would be added to gitignore or something? Instead, it discarded the changes, and to the user, it looked like VSCode did rm -rf and not that this was the behavior of git. Since the changes were never committed, even git reflog can't save them. permalink fedilink source parent hideshow 2 child comments replies: [–] Mad_Punda@feddit.org 30 points 2 years ago (3 children) From this issue: https://github.com/microsoft/vscode/issues/32459 It appears that the behavior actually included a git clean. Which is insane in my opinion. Not sure if they changed it since, but there’s definitely a dev defending it. permalink fedilink source parent hideshow 6 child comments replies: [–] MajorHavoc@programming.dev 12 points 2 years ago It appears that the behavior actually included a git clean. Which is insane in my opinion. Yeah. Building a convenient accessible context free way to run git clean...sure feels like the actions of someone who just wants to watch the world burn. permalink fedilink source parent [–] bamboo@lemmy.blahaj.zone 3 points 2 years ago (1 child) Yeah, it's unclear to me at the time if the dialogue box in the screenshot appeared when doing a select all operation, but it reads as though the OP dev didn't understand git, discarded their work, and got upset that it was an option. Realistically if the dialogue box appeared, I'm not sure there would be anything else the IDE could do to prevent the dev from themselves. Perhaps reject operations affecting 5000 files? But then you'll just have someone with the same issue for 4000 files. permalink fedilink source parent hideshow 2 child comments replies: [–] Mad_Punda@feddit.org 13 points 2 years ago* The issue I linked has a very good analysis of the UX issues and several suggestions for fixing these. They went with a minor iteration on the original message box, which not only includes a clearer message and the number of files affected, but also defaults to not touching untracked files (while preserving the option to delete untracked files as before). permalink fedilink source parent [–] Scoopta@programming.dev 2 points 2 years ago (1 child) He said they're not going to change it, just make the dialog a lot more clear and add a second button to it that will only do a reset without the clean. permalink fedilink source parent hideshow 2 child comments replies: [–] BatmanAoD@programming.dev 6 points 2 years ago The second button is actually a pretty major change! permalink fedilink source parent
[–] Mad_Punda@feddit.org 30 points 2 years ago (3 children) From this issue: https://github.com/microsoft/vscode/issues/32459 It appears that the behavior actually included a git clean. Which is insane in my opinion. Not sure if they changed it since, but there’s definitely a dev defending it. permalink fedilink source parent hideshow 6 child comments replies: [–] MajorHavoc@programming.dev 12 points 2 years ago It appears that the behavior actually included a git clean. Which is insane in my opinion. Yeah. Building a convenient accessible context free way to run git clean...sure feels like the actions of someone who just wants to watch the world burn. permalink fedilink source parent [–] bamboo@lemmy.blahaj.zone 3 points 2 years ago (1 child) Yeah, it's unclear to me at the time if the dialogue box in the screenshot appeared when doing a select all operation, but it reads as though the OP dev didn't understand git, discarded their work, and got upset that it was an option. Realistically if the dialogue box appeared, I'm not sure there would be anything else the IDE could do to prevent the dev from themselves. Perhaps reject operations affecting 5000 files? But then you'll just have someone with the same issue for 4000 files. permalink fedilink source parent hideshow 2 child comments replies: [–] Mad_Punda@feddit.org 13 points 2 years ago* The issue I linked has a very good analysis of the UX issues and several suggestions for fixing these. They went with a minor iteration on the original message box, which not only includes a clearer message and the number of files affected, but also defaults to not touching untracked files (while preserving the option to delete untracked files as before). permalink fedilink source parent [–] Scoopta@programming.dev 2 points 2 years ago (1 child) He said they're not going to change it, just make the dialog a lot more clear and add a second button to it that will only do a reset without the clean. permalink fedilink source parent hideshow 2 child comments replies: [–] BatmanAoD@programming.dev 6 points 2 years ago The second button is actually a pretty major change! permalink fedilink source parent
[–] MajorHavoc@programming.dev 12 points 2 years ago It appears that the behavior actually included a git clean. Which is insane in my opinion. Yeah. Building a convenient accessible context free way to run git clean...sure feels like the actions of someone who just wants to watch the world burn. permalink fedilink source parent
[–] bamboo@lemmy.blahaj.zone 3 points 2 years ago (1 child) Yeah, it's unclear to me at the time if the dialogue box in the screenshot appeared when doing a select all operation, but it reads as though the OP dev didn't understand git, discarded their work, and got upset that it was an option. Realistically if the dialogue box appeared, I'm not sure there would be anything else the IDE could do to prevent the dev from themselves. Perhaps reject operations affecting 5000 files? But then you'll just have someone with the same issue for 4000 files. permalink fedilink source parent hideshow 2 child comments replies: [–] Mad_Punda@feddit.org 13 points 2 years ago* The issue I linked has a very good analysis of the UX issues and several suggestions for fixing these. They went with a minor iteration on the original message box, which not only includes a clearer message and the number of files affected, but also defaults to not touching untracked files (while preserving the option to delete untracked files as before). permalink fedilink source parent
[–] Mad_Punda@feddit.org 13 points 2 years ago* The issue I linked has a very good analysis of the UX issues and several suggestions for fixing these. They went with a minor iteration on the original message box, which not only includes a clearer message and the number of files affected, but also defaults to not touching untracked files (while preserving the option to delete untracked files as before). permalink fedilink source parent
[–] Scoopta@programming.dev 2 points 2 years ago (1 child) He said they're not going to change it, just make the dialog a lot more clear and add a second button to it that will only do a reset without the clean. permalink fedilink source parent hideshow 2 child comments replies: [–] BatmanAoD@programming.dev 6 points 2 years ago The second button is actually a pretty major change! permalink fedilink source parent
[–] BatmanAoD@programming.dev 6 points 2 years ago The second button is actually a pretty major change! permalink fedilink source parent
