New hire commit:

Message: Visual update

Gitlab: Showing 27 changed files with 21628 additions and 12296 deletions

Do I kill him before or after teaching him?

  • 24
    Did he covert tabs to spaces?
  • 24
    Revert his commits.

    Forbid pushes for his account.

    Wait till he shows up.

    Then explain.
  • 30
    I only changed 1 line, I swear!
  • 12
    @C0D4 hmmmm 4460 is equal to 4460. From that, Sherlock Holmes could infer time, location, committer (hehe), motive and all the rest of the case.
  • 8
    Kill while teaching.
  • 2
    Teach first, kill later.
  • 5
    @cursee @NoMad hueaheu gonna teach by killing
  • 5
    just kill.. the teaching will be a waste of your time, because he'll be dead anyway
  • 5
    Why do people not check their diffs before committing? :(
  • 9
    @filthyranter because he is the master fucker frontend Jedi master, why would he?
  • 2
    @C0D4 maybe you changed the class name
  • 2
    @NoMad 😂😂
  • 3
    @bikki previous dev would have used tabs + had a habit of leaving white space at the end of every line.

    I pressed save and bam, No more tabs.
  • 1
    How did he get hired in the first place?
  • 3
    Did he change /n with /r/n or reverse? Worked with people on several projects where they kept doing that due to misconfigured git on Windows.
  • 4
    Teach once
    Remind twice
    Punish twice
    Fire once

    That's enough chances even for an intern.

    Edit: but obviously don't accept commits like that. wtf.
Add Comment