Beehaw
  • Communities
  • Create Post
  • search
    Search
  • Login
  • Sign Up
JPDev ( JPDev@programming.dev )  to Programmer Humor@programming.dev · 3 years ago

Review Please

programming.dev

message-square
46
link
fedilink
576

Review Please

programming.dev

JPDev ( JPDev@programming.dev )  to Programmer Humor@programming.dev · 3 years ago
message-square
46
link
fedilink
  • ursakhiin ( ursakhiin@beehaw.org ) 
    link
    fedilink
    arrow-up
    5
    ·
    3 years ago

    Human made changes is likely not what caused this image to occur.

    111 files with that kind of change count is most likely a dependency update. But could also be that somebody screwed up a merge step somewhere.

    • ErwinLottemann ( ErwinLottemann@feddit.de ) 
      link
      fedilink
      arrow-up
      8
      ·
      3 years ago

      you should meet my coworker. this is one week worth of work. and he still only commit once a week.

      • uis ( uis@lemm.ee ) 
        link
        fedilink
        arrow-up
        2
        ·
        3 years ago

        and he still only commit once a week.

        WHYYYY?

      • ulterno ( ulterno@lemmy.kde.social ) BannedBanned
        link
        fedilink
        English
        arrow-up
        0
        ·
        3 years ago

        Relatable

    • shiftymccool ( shiftymccool@programming.dev ) 
      link
      fedilink
      arrow-up
      2
      ·
      3 years ago

      The only way I see that is a dependency update is if you’re versioning your node_modules or which is generally a no-no

      • ursakhiin ( ursakhiin@beehaw.org ) 
        link
        fedilink
        arrow-up
        1
        ·
        3 years ago

        Many organizations vendor packages in the repo for a number of different reasons and languages. Not just for node.

    • ulterno ( ulterno@lemmy.kde.social ) BannedBanned
      link
      fedilink
      English
      arrow-up
      1
      ·
      3 years ago

      Or maybe their IDE had a different auto indent config and they saved it all, then committed it all without checking the diff or the status.

      • jjjalljs ( jjjalljs@ttrpg.network ) 
        link
        fedilink
        arrow-up
        3
        ·
        3 years ago

        You should have an agreed upon format that is enforced by cicd. Prettier, black, whatever.

        • ulterno ( ulterno@lemmy.kde.social ) BannedBanned
          link
          fedilink
          English
          arrow-up
          0
          ·
          3 years ago

          I do like the idea of mandating git clang-format as the Kate project has.
          That way the other devs don’t need to change their own IDE settings to comply.

Programmer Humor@programming.dev

programmer_humor@programming.dev

Subscribe from Remote Instance

Create a post
You are not logged in. However you can subscribe from another Fediverse account, for example Lemmy or Mastodon. To do this, paste the following into the search field of your instance: !programmer_humor@programming.dev

Welcome to Programmer Humor!

This is a place where you can post jokes, memes, humor, etc. related to programming!

For sharing awful code theres also Programming Horror.

Rules

  • Keep content in english
  • No advertisements
  • Posts must be related to programming or programmer topics
    • If the mod doesn’t find it funny, you’re banned. Ha-ha!.. For real: do not use the community for “statements”. There are other places for such content. Keep it chill and funny.
Visibility: Public
globe

This community can be federated to other instances and be posted/commented in by their users.

  • 1.26K users / day
  • 2.37K users / week
  • 5.27K users / month
  • 8.25K users / 6 months
  • 464 local subscribers
  • 34.1K subscribers
  • 1.65K Posts
  • 27.1K Comments
  • Modlog
  • mods:
  • Feyter ( Feyter@programming.dev ) 
  • adr1an ( anzo@programming.dev ) 
  • BurningTurtle ( BurningTurtle@programming.dev ) 
  • Pierre-Yves Lapersonne ( pylapp@programming.dev ) 
  • BE: 0.19.21
  • Modlog
  • Legal
  • Instances
  • Docs
  • Code