• filcuk@lemmy.zip
      link
      fedilink
      arrow-up
      0
      ·
      1 month ago

      Like, damn son, at least make a daily archive of your project??
      You have to expect things to go wrong, otherwise you have no one but yourself to blame.

    • Korne127@lemmy.world
      link
      fedilink
      arrow-up
      0
      ·
      1 month ago

      The person didn’t have any git repository; probably a new programmer that didn’t know how version control works and just clicked discard without understanding what that means in this situation

    • Mixel@feddit.org
      link
      fedilink
      English
      arrow-up
      0
      ·
      1 month ago

      I have heard things from another apprentice who just does not use version control at all and the only copies are on his laptop and on his desktop. He is also using node.js with only 1 class and doesn’t know about OOP (not sure if you even use that in js no clue 😅) and has one big file with 20k lines of code I have absolutely no clue how he navigates through it

      • LANIK2000@lemmy.world
        link
        fedilink
        arrow-up
        0
        ·
        edit-2
        1 month ago

        Ey! Reminds me of my middle-school years! I still can’t belive I made an entire game without a single class… Just storing info in arrays and writing in comments what location represents what data. But I was a literal child, too young to read guides or sit through “long” tutorials.

        I don’t want to sound too mean, but whenever I see anything similar at work, I wish that person get a job they’re actually good at. It’s fine and all that the company started hiring actual programmers to fix things, but the fact that the old crew still fucks shit up with senior privileges is a major grievance.

      • e8d79@discuss.tchncs.de
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        I know the type. Usually the kind of confident know-it-all who refuses to learn anything but delivers changes really quickly so management loves them. I had the misfortune to fix such a project after that ‘rock-star’ programmer left the company. Unfortunately the lack of professional standards in our industry allows people like that to continuously fail upwards. When I left the project they rehired them and let them design the v2 of the project we just fixed.

        • LANIK2000@lemmy.world
          link
          fedilink
          arrow-up
          0
          ·
          1 month ago

          Jesus, reminds me of a similar story. My gf once lost a job to someone who literally just pasted code into LLMs, also delivering quickly, even tho it was hot garbage. Anyhow, she spent a lot of her time fixing his shit and so her output went down. I hope that company burns to the ground with completely un manageable software.

        • AlexWIWA@lemmy.ml
          link
          fedilink
          English
          arrow-up
          0
          ·
          1 month ago

          We really need some kind of board like the one that controls the title for engineers.

        • MajorHavoc@programming.dev
          link
          fedilink
          arrow-up
          0
          ·
          1 month ago

          When I left the project they rehired them and let them design the v2 of the project we just fixed.

          Lol. Wow.

          And that is why I’ve been unable to work myself out of a job in all my long years as a developer.

        • RagingRobot@lemmy.world
          link
          fedilink
          arrow-up
          0
          ·
          1 month ago

          My company for the longest time had two engineers they would give all the new projects to. They would rush through some prototype code as fast as they could then management would bring in a new team to take the project over. The code was always garbage and crammed into one place. I kept getting new projects and instead of starting from a nice clean slate we always had to build on that garbage. It sucked so bad.

      • Blackmist@feddit.uk
        link
        fedilink
        English
        arrow-up
        0
        ·
        1 month ago

        Those are rookie numbers. I have at least a 35k one somewhere. More than one actually.

        People run their businesses on this.

        • fibojoly@sh.itjust.works
          link
          fedilink
          arrow-up
          0
          ·
          1 month ago

          I once landed a job at a small company doing a software for medical analysis labs all over the country. Software had been around for over ten years at this point. They had no source control. Nothing. Absolute nightmare.
          They were literally starting to use source control when I arrived.
          In 2015.

          • Blackmist@feddit.uk
            link
            fedilink
            English
            arrow-up
            0
            ·
            1 month ago

            The “source control” when I first started was all the code on a shared drive, to check out a file you copied it to your machine, and renamed the extension on the shared drive to your initials.

            When somebody edited without doing this there would be full blown meltdowns over lost work.

    • Maven (famous)@lemmy.zipOP
      link
      fedilink
      arrow-up
      0
      ·
      1 month ago

      This link was included in the post but I realize that “source” was probably not the best label for it. Updated to make it more clear.

    • eating3645@lemmy.world
      link
      fedilink
      English
      arrow-up
      0
      ·
      1 month ago

      Steps to Reproduce:

      1.Go near this fucking shit editor.

      2.Commit the deadly sin of touching the source control options.

      🤣

        • Miles O'Brien@startrek.website
          link
          fedilink
          English
          arrow-up
          0
          ·
          1 month ago

          Having done exactly 0 research, I going to assume it’s one of those “DO NOT PRESS OKAY UNLESS YOU ARE EXPERIENCED AND KNOW WHAT YOU ARE DOING” and someone went “pffft I know what I’m doing. click now what does this option do…”

          • bamboo@lemmy.blahaj.zone
            link
            fedilink
            English
            arrow-up
            0
            ·
            1 month ago

            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.

              • bamboo@lemmy.blahaj.zone
                link
                fedilink
                English
                arrow-up
                0
                ·
                1 month ago

                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.

                • Mad_Punda@feddit.org
                  link
                  fedilink
                  arrow-up
                  0
                  ·
                  edit-2
                  1 month 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).

              • MajorHavoc@programming.dev
                link
                fedilink
                arrow-up
                0
                ·
                1 month 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.

              • Scoopta@programming.dev
                link
                fedilink
                arrow-up
                0
                ·
                1 month ago

                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.

            • Hawke@lemmy.world
              link
              fedilink
              arrow-up
              0
              ·
              1 month ago

              “Changes” are not the same thing as “files”.

              I’d expect that files that are not in version control would not be touched.

              • MajorHavoc@programming.dev
                link
                fedilink
                arrow-up
                0
                ·
                1 month ago

                Yeah. That’s discussed in more detail in the code change that resulted from the issue report.

                It’s a ballsy move by the VSCode team to not only include git clean but to keep it after numerous issue reports.

                As others discussed in that thread, git clean has no business being offered in a graphical menu where a git novice may find it.

                That said, I do think the expanded warning mesage they added addresses the issue by calling my out that whatever git may think, the user is about to lose some files.

              • Pyro@programming.dev
                link
                fedilink
                arrow-up
                0
                ·
                edit-2
                1 month ago

                “Changes” encompass more than you think. Creating / Deleting files are also changes, not just edits to a file.

                If the change is an edit to a tracked file, “Discard Changes” will reverse the edit. If the change is a new untracked file, “Discard Changes” will remove it as intended.

                It can also be both at the same time, which is why VSCode uses “Changes” instead of “Files”.

                • candybrie@lemmy.world
                  link
                  fedilink
                  arrow-up
                  0
                  ·
                  1 month ago

                  If the change is a new untracked file

                  Wasn’t the issue that it deleted a bunch of preexisting untracked files? So old untracked files.

              • EleventhHour@lemmy.world
                link
                fedilink
                English
                arrow-up
                0
                ·
                edit-2
                1 month ago

                Apparently, it means changes to the directory structure and what files are in them, not changes within the files themselves. It really ought to be more clear about this.

          • josefo@leminal.space
            link
            fedilink
            arrow-up
            0
            ·
            1 month ago

            If the “changes” are all your files, discarding them for me means basically delete my files, you know, the ones you are trying to add.

            • T156@lemmy.world
              link
              fedilink
              English
              arrow-up
              0
              ·
              edit-2
              1 month ago

              At the same time, OP seems a layman, and might be coming from things like Microsoft Word, where “Discard all changes” basically means “revert to last save”.

              EDIT: After reading the related issues, OP may have also thought that “discard changes” was to uninitialise the repository, as opposed to wiping untracked files.

          • bleistift2@sopuli.xyz
            link
            fedilink
            English
            arrow-up
            0
            ·
            1 month ago

            Nowadays the warning even says that this cannot be undone. Maybe that wasn’t present in 1.15, though.

            • TopRamenBinLaden@sh.itjust.works
              link
              fedilink
              English
              arrow-up
              0
              ·
              1 month ago

              It was. If you go through the OP thread, one of the responses is a picture of the dialog window that this user clicked through saying, “these changes will be IRREVERSIBLE”.

              The OP was just playing with a new kind of fire (VSCodes Git/source control panel) that they didn’t understand, and they got burned.

              We all gotta get burnt at least once, but it normally turns us into better devs in the end. I would bet money that this person uses source control now, as long as they are still coding.

          • PostingPenguin@feddit.org
            link
            fedilink
            English
            arrow-up
            0
            ·
            1 month ago

            Nope. The scary warning is even screenshotted and used as an example in the post report discussion.

            It’s quite the fun read!

  • BougieBirdie@lemmy.blahaj.zone
    link
    fedilink
    English
    arrow-up
    0
    ·
    1 month ago

    Often times, people learn that the stove is hot by touching it.

    It’s easier to blame the stove than the person who touched it. But if you laugh when you watch it happen, you’re probably not laughing at the stove.

    • hawgietonight@lemmy.world
      link
      fedilink
      arrow-up
      0
      ·
      1 month ago

      This is my take on this. People blaming the software but the truth is that no software can be trusted. Make backups. Make mistakes and learn. Sometimes it will be painful, but those are the most valuable lessons

    • BatmanAoD@programming.dev
      link
      fedilink
      arrow-up
      0
      ·
      1 month ago

      The user clicked an option to “discard” all changes. They then got a very clear pop-up saying that this is destructive and cannot be undone (there’s a screenshot in the thread).

      • Peer@discuss.tchncs.de
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        I very much understand how one can think this would revert any changes done to files under version control but not delete the ones that are not. I believe this dialog has since been updated to explicitly state that fact.

        • BatmanAoD@programming.dev
          link
          fedilink
          arrow-up
          0
          ·
          1 month ago

          Yes, the dialog was changed, as part of this linked issue (and maybe again after that; this whole incident is very old). After reading some of the comments on that issue, I agree with the reasoning with some of the commenters that it would be less surprising for that menu option to behave like git reset --hard and not delete tracked files.

    • Beacon@fedia.io
      link
      fedilink
      arrow-up
      0
      ·
      1 month ago

      I don’t know anything about programming, i came here from /all, but it seems to me that a command that’s this permanently destructive warrants a second confirmation dialog message reminding the user that the files will be permanently deleted and not undoable

      • CaptDust@sh.itjust.works
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        They clicked discard changes, confirmed it, and the computer did as instructed. This operation is normally not so destructive as it only discards uncommitted changes to realign the local directory with the remote server. Unfortunately for user, it sounds like they have never committed a change, so realignment meant reverting to an empty folder.

      • Starbuck@lemmy.world
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        I think it’s important to know that this program is for code developers, and the issue here is with a tool called git. Git is like file saving on steroids, because on top of saving a single file, you save many changes to files in git, add a comment for why you made those changes, and share your changes across dozens of files with other developers.

        What this guy did was develop for many months after starting to use git, but he never actually committed the files. Then he asked for to reset everything back to the original state, something that I do multiple times a day, and it gave him a warning that original means original and you will lose everything. And he said do it anyways.

        • tyler@programming.dev
          link
          fedilink
          arrow-up
          0
          ·
          1 month ago

          No he asked for a discard after importing the project into VS Code. discard in git terms refers to git reset, not git clean. Even if he wanted to run a git reset then this version of VS Code would have run a git clean and deleted everything. Imagine he committed all 5000 files, but had a secret.json that he hadn’t committed. He didn’t add it to gitignore either. Running a git reset --hard will not delete this file, but the VS Code button did exactly that because it ran a git clean.

        • Semi-Hemi-Lemmygod@lemmy.world
          link
          fedilink
          English
          arrow-up
          0
          ·
          1 month ago

          That’s not a very good dialog box. He didn’t make any changes, so discarding them doesn’t sound like a problem.

          There should be a notice when you enable source control that this will permanently delete all existing files with a checkbox (checked by default) that says “Add existing files to source control.”

          • Pyro@programming.dev
            link
            fedilink
            arrow-up
            0
            ·
            1 month ago

            He wouldn’t have seen the “Discard Changes” button at all if source control wasn’t already setup (and detected by VSCode).

            No program will delete files either when you initialize source control.

            • LordPassionFruit@lemm.ee
              link
              fedilink
              arrow-up
              0
              ·
              1 month ago

              My sibling ran into this issue once. I’m not sure if it’s a setting or a default, but vscode would assume they were working in a blank repo until they made a commit.

              Sounds like this person had the project (without source control) in another IDE, tried out VSCode, and it assumed that it was all ‘changes’. I don’t use VSCode, do I can’t say for certain, but I know my sibling lost ~4 hours of project set up for the same reason (though they immediately realized it was their fault).

              • Pyro@programming.dev
                link
                fedilink
                arrow-up
                0
                ·
                1 month ago

                Reading your comment and #32459, I realize that VSCode source control did have some major issues back then.

                It looks like they have improved though, as the latest VSCode I use doesn’t auto-initialize repositories anymore.

        • Beacon@fedia.io
          link
          fedilink
          arrow-up
          0
          ·
          1 month ago

          Hm ok yeah, that seems quite scary sounding so that i would strongly hesitate before clicking on “discard ALL changes”. Still, I wonder if a second confirmation dialog with more information is warranted for a command that’s so destructive.

            • mark3748@sh.itjust.works
              link
              fedilink
              arrow-up
              0
              ·
              1 month ago

              Then you don’t understand git or source control in general. If you don’t commit your changes, then you discard any changes, you’re set back to the last commit. Discarding changes also includes any un-committed new files.

              • onlinepersona@programming.dev
                link
                fedilink
                English
                arrow-up
                0
                ·
                1 month ago

                Which is exactly the situation the dude was in. As a newbie, it’s an easy mistake to make. Telling somebody who doesn’t know “well, would you look at that, you didn’t know!” is not just unhelpful, it’s useless and condescending.

                Anti Commercial-AI license

              • tyler@programming.dev
                link
                fedilink
                arrow-up
                0
                ·
                1 month ago

                discarding changes does not discard uncommitted new files. The VS Code button did a git clean which is completely unexpected. Git even refers to a git clean with completely different terminology.

                git reset -> “Resets the index and working tree. Any changes to tracked files in the working tree since are discarded.

                git clean -> “Cleans the working tree by recursively removing files that are not under version control, starting from the current directory.”. This command also requires you to specify a force option to actually do something, else it quits with an error.

                Note that git clean never once refers to discarding anything, and git reset never refers to removing untracked files. VS Code was doing an idiotic thing. Running git reset --hard AND git clean. There is absolutely no reason to be running git clean from an UI button ever. If you want to remove a file you can explicitly remove it.

                Imagine that the button said “Discard all changes” and then it ran rm -rf --no-preserve-root /*. Would that make sense as a button? No. It definitely would not.

            • subignition@fedia.io
              link
              fedilink
              arrow-up
              0
              ·
              1 month ago

              It’s changes from the prior commit in the repository, which, if they had not committed anything prior, would have been an empty directory.

              This is perhaps a good lesson in teaching version control as its own concept rather than “streamlining it” by bundling it with an editor.

              • conciselyverbose@sh.itjust.works
                link
                fedilink
                arrow-up
                0
                ·
                1 month ago

                You shouldn’t be taking ownership of files and then deleting them without communication a hell of a lot better than that.

                I understand what happened. I’m saying that if you’re going to delete stuff that was there before the software was, your flow to adding a project should include suggesting a base level commit of everything that’s there already.

                • subignition@fedia.io
                  link
                  fedilink
                  arrow-up
                  0
                  ·
                  1 month ago

                  That’s definitely fair, creating a repository in a non-empty directory could definitely suggest auto-committing the current state if it doesn’t already. I don’t use VSCode so I wouldn’t know.

                  Although now that I think about it, that could have been the intention here but not automatic, if that’s why 5k+ files were staged without the user explicitly staging them. Extra tragic if that’s the case.

              • Scubus@sh.itjust.works
                link
                fedilink
                arrow-up
                0
                ·
                1 month ago

                Ok then, the changes to the repository shouldve been discarded. Anything he uploaded shouldve been deleted from the server. Why were files on his local machine deleted?

                • subignition@fedia.io
                  link
                  fedilink
                  arrow-up
                  0
                  ·
                  1 month ago

                  What makes you think a server was involved here? It was a local repository, evidenced by the reporter’s bewilderment that files can be deleted without going to the Recycle Bin first. Which tells us that in addition to VCS, they were unfamiliar with Windows as well.

            • Pyro@programming.dev
              link
              fedilink
              arrow-up
              0
              ·
              1 month ago

              It’s not that. It means discard all changes made after the last change committed to this local repository.

          • TeamAssimilation@infosec.pub
            link
            fedilink
            arrow-up
            0
            ·
            1 month ago

            I guess cancelling would go back to the “Then you want to commit all files?” dialog, which the user didn’t want to, he just wanted to cancel whatever the IDE was trying to start.

      • InternetCitizen2@lemmy.world
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        Perhaps. Still I am not sure why someone who is not aware of this would be using VSC. If they are a student then what kind of project are they working on that they have so many files?

      • ArbiterXero@lemmy.world
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        The problem is that these are “source control basics” that everyone needs to learn the hard way once it seems.

        Waiting 3 months in between commits however is a really bad rookie mistake because you were worried about making a commit that wasn’t perfect.

          • MajorHavoc@programming.dev
            link
            fedilink
            arrow-up
            0
            ·
            1 month ago

            Interesting. I wouldn’t know, because I code everything perfectly the first time.

            Disclaimer: The above flagrant lie was brought to you by my also using rebase and squash to hide all of my mistakes.

        • tyler@programming.dev
          link
          fedilink
          arrow-up
          0
          ·
          1 month ago

          the problem is that VS Code ran git clean if you clicked yes, which is completely idiotic behavior. No other git client, text editor, or IDE on the planet does that.

  • unalivejoy@lemm.ee
    link
    fedilink
    English
    arrow-up
    0
    ·
    1 month ago

    Say you don’t know how to use git without saying you don’t know how to use git.

    • ChaoticNeutralCzech@feddit.org
      link
      fedilink
      English
      arrow-up
      0
      ·
      1 month ago

      That’s what happens when people stumble across that website called GitHub, get hooked and now have unrealistic expectations for the real git.

      “I just installed Git for Windows. Where is the drag-to-upload box?”

      — A statement dreamt up by the utterly deranged

      Real git involves a lot of sweat, requires you to clean up any mess you make, and communicate with any partners about their preferred techniques instead of rawdogging it and waiting for issues. The pushing and pulling will come naturally but you need to know how and when to release, and be clear about how you wish to commit. People might judge you for using the word “master” but it should be alright in private.

      • colonelp4nic@lemmy.world
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        I’m literally a software dev working for a top company and I can barely use git on the CLI. I do all of my version control operations using a GUI, so there’s no sense in gatekeeping any of that. This is true of both my work projects and personal ones. It’s cool if you prefer the CLI, but it is absolutely not a required skill in order to have a successful and meaningful career.

        • mark3748@sh.itjust.works
          link
          fedilink
          arrow-up
          0
          ·
          1 month ago

          I agree that the gatekeeping isn’t a good thing, but you should learn at least the basics of the CLI. It will give you a better understanding of what’s going on behind your GUI and makes troubleshooting and fixing problems a lot easier.

          Definitely not required but it is absolutely a skill worth having.

          • Azzu@lemm.ee
            link
            fedilink
            arrow-up
            0
            ·
            1 month ago

            It’s absolutely not a skill worth having. If you ever run into issues and need the CLI, you can always get your knowledge right in that moment. If you already can do everything with your GUI and get the same results, getting the knowledge to do it some other way is just wasted time and duplicate work.

      • 4am@lemm.ee
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        involves a lot of sweat, requires you to clean up any mess you make, and communicate with any partners about their preferred techniques instead of rawdogging it and waiting for issues. The pushing and pulling will come naturally but you need to know how and when to release, and be clear about how you wish to commit. People might judge you for using the word “master” but it should be alright in private.

        Don’t talk about my mom that way

      • BearOfaTime@lemm.ee
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        People might judge you for using the word “master” but it should be alright in private.

        I snorted. It was my inner 12-year-old’s fault. (Also because of recently some idiots getting up in arms about these terms in technology.)

        • Prandom_returns@lemm.ee
          link
          fedilink
          arrow-up
          0
          ·
          edit-2
          1 month ago

          “up in arms”:

          Reality:
          – “just don’t use them, some people find them offensive”
          – “ok”

          Anonymous techbros online:
          “yOu CanT sAY aNYtHiNg ThEsE daYs”

    • Kaeru@slrpnk.net
      link
      fedilink
      English
      arrow-up
      0
      ·
      1 month ago

      Alright you convinced me its time to pick up this skill. How does one best learn git? Just play around with it and break things?

      • LordPassionFruit@lemm.ee
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        That’s basically how I did it.

        To properly learn it using this method, create a directory that contains only text files and sub directories and treat it like a real project. Add files, delete them, play around with updating the repository. Try and go back a few updates and see how the things react. Since it’s not a real project there’s no risk of loss, but you’ll still get to see the effects of what you do.

        • mEEGal@lemmy.world
          link
          fedilink
          arrow-up
          0
          ·
          1 month ago

          that’s a necessary step in your learning process, but certainly not sufficient. I’d recommend reading the book, since it shows in greut detail the inner workings of Git along with the basic concepts :

          https://git-scm.com/book/en/v2

    • bjorney@lemmy.ca
      link
      fedilink
      arrow-up
      0
      ·
      1 month ago

      Git doesn’t automatically recursively add all files in the directory to the repository though - VSCode decided that should be the default behavior, while other editors (intellij) ask if you want to add newly created files to version control

  • Treczoks@lemmy.world
    link
    fedilink
    arrow-up
    0
    ·
    1 month ago

    Someone who does not know about “permanent delete” and not having backups, especially when switching to a new system, should have no business complaining about this.

  • riodoro1@lemmy.world
    link
    fedilink
    arrow-up
    0
    ·
    1 month ago

    Typical web developer. He didn’t even know files can be deleted without going into „recycle bin”

      • x00z@lemmy.world
        link
        fedilink
        English
        arrow-up
        0
        ·
        1 month ago

        I think it’s a joke about how noobs only learn javascript and make blazing fast webapps while knowing nothing about computers.

  • onlinepersona@programming.dev
    link
    fedilink
    English
    arrow-up
    0
    ·
    1 month ago

    The reactions here are why people don’t join forums, don’t ask questions, or choose to learn alone. “duh, I knew that”. Yes, the dude didn’t, which is exactly why he’s frustrated. I think too many have forgotten what it’s like to be a beginner and make a fatal mistake, which would explain the mocking responses here and things like recommending new linux users Arch.

    Anti Commercial-AI license

    • Zagorath@aussie.zone
      link
      fedilink
      English
      arrow-up
      0
      ·
      1 month ago

      I understand the impulse to be empathetic and kind. But it’s very hard to respond in good faith to someone who just made a post where more than half the words are “fuck you”.

      • madcaesar@lemmy.world
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        A feature that permanently deletes 5000 files with one click without warning deserves a fuck you.

          • tfw_no_toiletpaper@lemmy.world
            link
            fedilink
            arrow-up
            0
            ·
            1 month ago

            OK this is hilarious

            When you sell hammers you’ll likely have people using them to hit their own heads, which, understandably, they will put the hammer at fault. Now, we already put a big don’t hit this on your own head label on our hammer. Should we actually prohibit people from head hitting with our hammers? Probably not, since some users still want to hit heads with it. It’s just how hammers work.

          • Prandom_returns@lemm.ee
            link
            fedilink
            arrow-up
            0
            ·
            1 month ago

            “Discard changes” is usually equivalent to “cancel” or “quit without saving”. Not shift+delete files lol.

          • Buddahriffic@lemmy.world
            link
            fedilink
            arrow-up
            0
            ·
            1 month ago

            I disagree that that warning is reasonably clear. Even the comment that included it has the line of thought, where the user, not knowing what terms git uses thinks that they just did an action that is going to change each of their files. It makes sense that they’d want to discard those changes. That user then goes on with some snark about not wanting to learn any more about what they are playing with and that other programs would do the same, but “discard changes” seems like it would have a clear meaning to someone who doesn’t know git.

            The warning says it isn’t undoable but also doesn’t clarify that the files themselves are the changes. Should probably have a special case for if someone hits discard changes on a brand new repository with no files ever checked in and hits discard on a large number of files instead of checking them in. Even a “(This deletes all of the local files!)” would make it clear enough to say what the warning is really about.

          • ClassifiedPancake@discuss.tchncs.de
            link
            fedilink
            arrow-up
            0
            ·
            1 month ago

            If you have no idea what Git is, that warning message is not telling you you’re about to delete 5000 files.

            But I wonder if this person maybe does know about Git because they used the word „stage“.

    • CosmicTurtle0@lemmy.dbzer0.com
      link
      fedilink
      English
      arrow-up
      0
      ·
      1 month ago

      There is a difference between someone who is new and experiences something like their IDE deletes a file that was unexpected and asking a question about why it did that.

      Then there are arrogant assholes who believe their shit doesn’t stink and that they couldn’t have done anything wrong and it was the IDE’s fault for not knowing what they wanted to do versus what they commanded it to do.

      The OP is the latter.

      • rothaine@lemm.ee
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        I mean, not entirely, and he says he lost months worth of work. Like imagine you know nothing of git:

        • Click buttons in the IDE to add source control.

        • IDE says a bunch of files have been changed.

        • But I don’t want to make changes to the files, I want to source control them.

        • Attempt to undo the changes. Click “discard changes” thinking it will put them back to how they were before clicking add source control. Get a warning dialog that this is not undoable, but that’s fine because I don’t want whatever changes it made to my files anyway.

        • All files are deleted and unrecoverable.

        Like that experience sucks balls and it’s reasonable that a person wouldn’t expect “discard” == “delete”. Also, from reading the GitHub thread, apparently at that time VSCode was doing a git clean when you clicked this. Which like…yeah why the hell would it do that lol? I don’t think I have ever used git clean in my entire career.

        • Scary le Poo@beehaw.org
          link
          fedilink
          arrow-up
          0
          ·
          1 month ago

          Months worth of work. Going an entire day without committing should never happen. Also, rawdogging it without a backup?

          Nope, dude learned a hard lesson. No sympathy. He thought that the rules of data storage don’t apply to him and he got boned.

          • rothaine@lemm.ee
            link
            fedilink
            arrow-up
            0
            ·
            1 month ago

            He did learn a hard lesson, but it sounds like this was exactly what he was attempting to rectify. “This project is pretty important to me. I should probably figure out how to use source control and put it on GitHub” but then by doing so, and due to some arguably poor UX decisions in VSC, destroyed the project.

            If someone’s trying to learn how to do something you can’t just be like “well you should’ve already known how”, you know what I mean?

            Well actually, let’s qualify that: there are cases where people try to jump right in the deep end. “I’m just learning woodworking! Step 1: build a new deck for my house”, like yeah bro what are you doing, let’s start with a birdfeeder or something.

            But that’s not what happened here. He tried to use the built-in GUI, which many would assume would be easier to learn than jumping right to the CLI, and it burned him in a way he didn’t even realize was possible.

  • AnAmericanPotato@programming.dev
    link
    fedilink
    English
    arrow-up
    0
    ·
    1 month ago

    I feel bad for this kid. That really is a bad warning dialog. Nowhere does it say it’s going to delete files. Anyone who thinks that’s good design needs a break.

    Half the replies are basically “This should be obvious if your past five years of life experience is similar to mine, and if it isn’t then get fucked.” Just adding insult to injury.

    • Scary le Poo@beehaw.org
      link
      fedilink
      arrow-up
      0
      ·
      edit-2
      1 month ago

      If you’re going to use a git tool, you need to know how git works.

      There are 0 excuses for not having months of work in a repo, none. I have no sympathy whatsoever. How the fuck do you spend so many months without backing up your project or stuffing it in a repo?

      No sympathy. Dude is a shit developer and he learned an invaluable lesson.

      • JackbyDev@programming.dev
        link
        fedilink
        English
        arrow-up
        0
        ·
        1 month ago

        If you’re going to use a git tool, you need to know how git works.

        I guarantee you at least half of git users would get glossy eyes as soon as you mention blobs and trees, yet they all still manage to use it daily successfully.

        There are 0 excuses for not having months of work in a repo, none. I have no sympathy whatsoever. How the fuck do you spend so many months without backing up your project or stuffing it in a repo?

        I need you to listen to me very carefully: THEY WERE FUCKING SETTING UP A REPO WHEN THIS HAPPENED.

      • AnAmericanPotato@programming.dev
        link
        fedilink
        English
        arrow-up
        0
        ·
        1 month ago

        My guess is that this is a teenager, and this is probably their first experience with git and version control in general. Just a hunch.

        Anyway, it is reasonable to expect a mainstream GUI app from one of the largest companies in the world to be approachable for people who do not know all the inner workings of the command line tools that are used behind the scenes. And it is reasonable to expect any destructive action to have clear and bold warnings. “Changes will be discarded” is not clear. What changes? From the user’s perspective, the only changes were regarding version control, so “discarding” that should leave them where they started — with their files intact but not in version control.

        Have mercy on the poor noobs. We were all there once.

    • JackbyDev@programming.dev
      link
      fedilink
      English
      arrow-up
      0
      ·
      1 month ago

      It’s so fucking infuriating that so many devs act like this. “This should’ve been obvious!” Fuck off, that’s an unhelpful statement. “You should’ve been using version control! No backup, no sympathy!” Fuck off, they were literally trying to begin using version control for backups.

      Even half the comments on this very Lemmy thread are disparaging this dev. I wonder how many actually read the thread and found that there was a bug discovered causing this feature to delete files not even associated with git?

      But, congratulations to them, I suppose. Congratulations on making fun of someone. I hope it makes them feel powerful. 🙄 Devs can be so toxic.

    • Omega_Jimes@lemmy.ca
      link
      fedilink
      arrow-up
      0
      ·
      1 month ago

      I’m not great at English, but “discard all changes” shouldn’t ever mean “Delete”.

      • Michal@programming.dev
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        In the context of version control it does. Discarding a change that creates a file means deleting the file.

        • thebestaquaman@lemmy.world
          link
          fedilink
          arrow-up
          0
          ·
          1 month ago

          If you have set up your staging area for a commit you may want to discard (unstage) changes from the staging area, as opposed to discarding changes in the working directory.

          Of course, the difference between the two is obvious if you’re using git CLI, but I can easily see someone using a GUI (and that maybe isn’t too familiar with git) misunderstanding “discard” as “unstage”.

          Either way, what happened here indicates that all the files were somehow added to the VC, without having been committed first, or something like that, because git will not let you discard a file that is untracked, because that wouldn’t make any sense. The fact that the GUI let this person delete a bunch of files without first committing them to the index is what makes this a terrible design choice, and also what makes the use of the word “discard” misleading.

        • Omega_Jimes@lemmy.ca
          link
          fedilink
          arrow-up
          0
          ·
          1 month ago

          Ok fair enough, but I’m under the impression these files existed before the source control was implemented.

          I guess it’s all up to how the program handles existing files.

          • FiskFisk33@startrek.website
            link
            fedilink
            arrow-up
            0
            ·
            1 month ago

            I guess the newly created git repository was empty, and all the files that was present in the folder represented “changes”

      • stebo@lemmy.dbzer0.com
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        I’m pretty sure vscode shows a confirmation dialog when discarding changes will permanently delete a file. I’ve done that recently with temporary files that were no longer needed.

    • cocobean@bookwormstory.social
      link
      fedilink
      English
      arrow-up
      0
      ·
      1 month ago

      Also, why not send them to the recycle bin? I never really thought about it before, but that does seem a reasonable UX improvement for this case

      • stetech@lemmy.world
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        Because “the underlying Git nukes them right away, so why shouldn’t we perma-delete the files, too?”

        Anything else’d be effort…

      • murtaza64@programming.dev
        link
        fedilink
        arrow-up
        0
        ·
        1 month ago

        I wonder if there’s already a git extension to automatically stash the working tree on every clean/reset/checkout operation…

    • kehet@sopuli.xyz
      link
      fedilink
      arrow-up
      0
      ·
      1 month ago

      Came here to say this. No one deserves this, not even new programmers who try to learn things.

      Some programming tools are really powerful compared to what new users are used to. If you come from the world of Microsoft Office and Apple whatever it’s called, everything is saved automatically to cloud and there is some local backup file somewhere which you can just restore. Modern programs are designed to protect users against their own mistakes, and when suddenly that is taken away, it can be a jarring experience.