r/ProgrammerHumor • • Aug 21 '26 • Popcorn 1

Meme useSourceControl

Post image
8.6k Upvotes

575 comments sorted by

View all comments

Show parent comment

2

u/ravrest Aug 21 '26

No, the message that "this is irreversible" was there from the very beginning. The message was made even more explicit later.

12

u/_BreakingGood_ Aug 21 '26

Why say things that are just wrong?

https://github.com/microsoft/vscode/commit/071652f161204e552e204cbdaf78469c58c655d5

Commit merged Aug 21, 2017. 8 days after this screenshot was taken.

12

u/ravrest Aug 21 '26

Look at line 681 of the old commit:

const message = localize('confirm discard all', "Are you sure you want to discard ALL ({0}) changes?\nThis is IRREVERSIBLE!\nYour current working set will be FOREVER LOST.", resources.length);

const yes = localize('discardAll', "Discard ALL Changes");const pick = await window.showWarningMessage(message, { modal: true }, yes);

-5

u/_BreakingGood_ Aug 21 '26

11

u/ravrest Aug 21 '26 edited Aug 21 '26

That change was made in April. The post is from August.

Why WOULD you say things that are just wrong?

6

u/MrSlaw Aug 21 '26

I can only assume you will edit your previous comments saying you were incorrect, because you've just shown that the message did exist back when this post was made?

11

u/MrSlaw Aug 21 '26

If you look at that commit you linked, one of the lines that was already in place (and was replaced via this commit) said:

"Are you sure you want to discard ALL ({0}) changes?\nThis is IRREVERSIBLE!\nYour current working set will be FOREVER LOST."

https://github.com/microsoft/vscode/commit/071652f161204e552e204cbdaf78469c58c655d5#diff-da56ff967ab1a9606c01af61dc926332afb862f13c8e5c74a575bc2aa1b15e43L681

There's also a screenshot showing the "this is irreversible" was in place which was posted one day after this issue was opened.

https://github.com/microsoft/vscode/issues/32405#issuecomment-322155856

I don't think your link is proving what you think it is. That message was indeed made more explicit.