4 ms·

You fired for someone finding bugs in your code? This is beyond ridiculous and you're creating a hostile environment for your developers. It'll come back and b

You fired for someone finding bugs in your code? This is beyond ridiculous and you're creating a hostile environment for your developers. It'll come back and bite you in the future.

Cthulhu_ 9y ago
He was fired for doing an unsolicited code review, which the grandparent did not ask for nor deemed necessary. Plus the wording implies the grandparent felt attacked - as the thread earlier mentioned, you shouldn't barge in and go all Steve Jobs and yell THIS IS WRONG! THIS SHOULD BE FIXED! right off the bat. I do understand it though, and when you're fresh in a project it's easy to go for the low hanging fruit like that. But, be subtle about it, ask if it's welcome, that kinda thing.
- toomanybeersies 9y ago
  What the hell is an unsolicited code review? When you start work with a new codebase you spend some time to get acquainted with it. It only makes sense to take notes of bugs or possible improvements when you do it. A pair of fresh eyes will always find improvements to be made.
  - aaomidi 9y ago
    This so much. If you expect me to take on a project without looking at the previous code then you're simply a fool.
    - toomanybeersies 9y ago
      I really wish companies let you have a look at the codebase before you started. Like, sign an NDA and give you 15 minutes to look at it. It would've made me reconsider the job I currently have. Upon saying that, I am enjoying the challenge of trying to eliminate as much tech debt as possible.
- iajrz 9y ago
  I see your point, and I agree that it's good to be nice to people. But being in leadership and spending time around a few hundred coders over the hears has taught me the value of someone who does this. I hope this wasn't the reason the person was fired, because it's a missed opportunity for management: someone with that kind of drive and commitment to quality is valuable to have - and hard to find. I can count with my hands the number of developers with that kind of drive I've found. With no further context - meaning: I'm making a lot of assumptions here - I think I'd've helped them schedule time to evaluate these opportunities for improvement (how much do they cost to fix? how much will they cost if not fixed? How can we prevent this from getting in new code? etc.), and when the numbers are right - schedule time for fixing them, too. If non-agreeableness was their sole defect - it could've been worked on over time. That's what leaders are there for: to help their collaborators grow.
- nickthemagicman 9y ago
  Grandparent?
  - majewsky 9y ago
    Parent = the comment that this comment replies to. Grandparent = the parent of the parent.
    - mfoy_ 9y ago
      Often abbreviated as "GP".
    - deleted 9y ago
      [deleted]
    - nickthemagicman 9y ago
      Thank you!Neverheard of it referred to that way before but it totally makes sense!
  - coatmatter 9y ago
    Why many of us do this: It's often easier to refer to previous posts relatively rather than to cite usernames. For "OP", others will know to simply look (or think) back up to the top (or thereabouts). For "parent", it's usually the comment above the one that is being replied to. So parent to your post is what I'm referencing, but I'm now replying to you (and have taken no notice of the usernames because it's easier to focus on the content).
    - kaybe 9y ago
      One also often sees the same line of argument continued by other posters, so while the conversation is consistent with two (or more) sides being explored, the posters change. I really like this about HN - it really is a group conversation most of the time.
- deleted 9y ago
  [deleted]
- holtalanm 9y ago
  'unsolicited code review'??? dafuq is that? The more eyes on a piece of code, the better.

You fired for someone finding bugs in your code? This is beyond ridiculous and you're creating a hostile environment for your developers. It'll come back and bite you in the future.

Cthulhu_ 9y ago

He was fired for doing an unsolicited code review, which the grandparent did not ask for nor deemed necessary. Plus the wording implies the grandparent felt attacked - as the thread earlier mentioned, you shouldn't barge in and go all Steve Jobs and yell THIS IS WRONG! THIS SHOULD BE FIXED! right off the bat. I do understand it though, and when you're fresh in a project it's easy to go for the low hanging fruit like that. But, be subtle about it, ask if it's welcome, that kinda thing.

toomanybeersies 9y ago
What the hell is an unsolicited code review? When you start work with a new codebase you spend some time to get acquainted with it. It only makes sense to take notes of bugs or possible improvements when you do it. A pair of fresh eyes will always find improvements to be made.
- aaomidi 9y ago
  This so much. If you expect me to take on a project without looking at the previous code then you're simply a fool.
  - toomanybeersies 9y ago
    I really wish companies let you have a look at the codebase before you started. Like, sign an NDA and give you 15 minutes to look at it. It would've made me reconsider the job I currently have. Upon saying that, I am enjoying the challenge of trying to eliminate as much tech debt as possible.
iajrz 9y ago
I see your point, and I agree that it's good to be nice to people. But being in leadership and spending time around a few hundred coders over the hears has taught me the value of someone who does this. I hope this wasn't the reason the person was fired, because it's a missed opportunity for management: someone with that kind of drive and commitment to quality is valuable to have - and hard to find. I can count with my hands the number of developers with that kind of drive I've found. With no further context - meaning: I'm making a lot of assumptions here - I think I'd've helped them schedule time to evaluate these opportunities for improvement (how much do they cost to fix? how much will they cost if not fixed? How can we prevent this from getting in new code? etc.), and when the numbers are right - schedule time for fixing them, too. If non-agreeableness was their sole defect - it could've been worked on over time. That's what leaders are there for: to help their collaborators grow.
nickthemagicman 9y ago
Grandparent?
- majewsky 9y ago
  Parent = the comment that this comment replies to. Grandparent = the parent of the parent.
  - mfoy_ 9y ago
    Often abbreviated as "GP".
  - deleted 9y ago
    [deleted]
  - nickthemagicman 9y ago
    Thank you!Neverheard of it referred to that way before but it totally makes sense!
- coatmatter 9y ago
  Why many of us do this: It's often easier to refer to previous posts relatively rather than to cite usernames. For "OP", others will know to simply look (or think) back up to the top (or thereabouts). For "parent", it's usually the comment above the one that is being replied to. So parent to your post is what I'm referencing, but I'm now replying to you (and have taken no notice of the usernames because it's easier to focus on the content).
  - kaybe 9y ago
    One also often sees the same line of argument continued by other posters, so while the conversation is consistent with two (or more) sides being explored, the posters change. I really like this about HN - it really is a group conversation most of the time.
deleted 9y ago
[deleted]
holtalanm 9y ago
'unsolicited code review'??? dafuq is that? The more eyes on a piece of code, the better.