Do you sometimes feel weird commenting on a more Senior developer merge requests?

Do you sometimes feel weird commenting on a more Senior developer merge requests?

CRIME Shirt $21.68

Tip Your Landlord Shirt $21.68

CRIME Shirt $21.68

  1. 2 weeks ago
    Anonymous

    Yes

  2. 2 weeks ago
    Anonymous

    The seniors at the company I work at don't get reviewed. They commit the worst shit you've ever seen but they're tight with the team leads so nobody can complain

  3. 2 weeks ago
    Anonymous

    no why? did you post cringe?

    • 2 weeks ago
      Anonymous

      >Unable to read
      IQfy at it's finest

  4. 2 weeks ago
    Anonymous

    You can't imagine. It's hard. And it also sucks that often they see me as a senior.

  5. 2 weeks ago
    Anonymous

    The senior developers are all moronic. Usually I don't comment on it because I know they're not going to listen to or even understand my feedback.

  6. 2 weeks ago
    Anonymous

    Don't do it, since I am a senior and don't know shit

  7. 2 weeks ago
    Anonymous

    No because I got downleveled once at gayman due to “startup” background and “seniors” were moronic.

  8. 2 weeks ago
    Anonymous

    As a senior dev when I assign a non-trivial PR to someone and I get "looks good, approved." Then I wonder why I even bother. This is one of the few forms of technique sharing and collaboration that we have and you just want to pass it over as an attendance check.

    • 2 weeks ago
      Anonymous

      I don't expect you to value any honest opinion I would provide. In fact I expect you to dislike me for it.
      Even if you tell me to offer you honest feedback on code reviews, 99% of people who claim to want honest feedback do not, so I will not. Sorry.

  9. 2 weeks ago
    Anonymous

    I used to do it, but they sperg out as a defense mechanism so now its just lgtm
    gotta choose your battles

  10. 2 weeks ago
    Anonymous

    >Merge requests
    lol, lmao
    >Pushes to master

  11. 2 weeks ago
    Anonymous

    >Do you sometimes feel weird commenting on a more Senior developer merge requests?
    No and I am not even a developer. "Stop putting private keys in the code repo. This is a security incident. See the incident manager."

  12. 2 weeks ago
    Anonymous

    no because im not an insecure trainwreck

  13. 2 weeks ago
    Anonymous

    Yes, and also the juniors

    t. moron senior outclassed by his underlings

    • 2 weeks ago
      Anonymous

      Don't feel bad, that's how things are supposed to be.

  14. 2 weeks ago
    Anonymous

    I would never do it.
    >What if it introduces a bug?
    Not my problem.
    Why should I worsen my relationship with the seniors and be seen as a snitch?
    Let it fail, and then they will be blamed and I will replace them, while also being the nice non-confrontational guy.

    • 2 weeks ago
      Anonymous

      >while also being the nice non-confrontational guy.
      a doormat with no spine

      • 2 weeks ago
        Anonymous

        doormats always win in the end
        guys with spines are at each others throats and at some point this will block you
        look at literally every CEO, just a soulless bot following orders

        • 2 weeks ago
          Anonymous

          bro you are too black and white for me

          • 2 weeks ago
            Anonymous

            I'm not black or white
            I'm green
            I go through this hell called a job just for money
            I literally dgaf about anything else and will do exactly what is required to make more
            I don't care about being viewed as having a spine, I don't care about friends, my pride or any of that bullshit
            I just want to bury everyone else and come out on top
            The minute society forced me to go through this hell is the minute everyone became my sworn enemy
            Beware the rage of the NEET forced to work

          • 2 weeks ago
            Anonymous

            k

  15. 2 weeks ago
    Anonymous

    What’s PR? Pull request? Push request? Git? Penis rush?

    • 2 weeks ago
      Anonymous

      Penile Rectum. It's a very serious deformity.

  16. 2 weeks ago
    Anonymous

    code review is an incredible waste of time for most companies.
    so much time wasted, and you still probably have bugs out the ass.
    people always take it personally, no matter what they say or what the "team culture" is. everyone hates being criticized.

  17. 2 weeks ago
    Anonymous

    No.

    My team lead (company CTO)'s PRs I review just as fierce as everyone else's. I often have the most feedback and change requests on his PRs.

    Get in line, fricker. You're one of us. Act like it. We're building quality software over here, and I am NOT going to let you degrade the stability of our software with your hacky workarounds and "quick fixes".

    Luckily he appreciates it.

  18. 2 weeks ago
    Anonymous

    You jr weenies shouldn't feel weird commenting on my reviews. The point of code review is to solicit feedback. What you should be worried about is engaging in futile arguments with my feedback on your reviews. All else equal- and it virtually always is- I'm right and you're wrong. My feedback is best for the long-term health of the project, which it's my job to oversee.

Your email address will not be published. Required fields are marked *