Ranter
Join devRant
Do all the things like
++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatar
Sign Up
Pipeless API
From the creators of devRant, Pipeless lets you power real-time personalized recommendations and activity feeds using a simple API
Learn More
Comments
-
VaderNT16345yWell, the dev is childish. He/she immediately felt attacked when you stated facts and only referenced things (the commit, the failing build) instead of people. And the excuses make it clear the dev knew he/she did something wrong.
Is this normal or ok? Yes, it's somewhat prevalent in the industry. No, it's not okay. -
Builds from development branches should NOT be failing. Make changes on your own branch, rebase from development branch, test if build works in CI (if that exists), merge with development branch (optionally: create a merge request that someone else merges for you)
-
wypaliz235y@kescherRant yeah good point, I usually point feature branches to Dev env... But even that won't help at this point because of how jacked the architecture is. Or, the lack of an architecture plan might be the better explanation.
-
wypaliz235y@VaderNT to be fair, I've been pointing out these types of things for the past two days and they are all tired of me. I called them out yesterday for making changes directly in the release branch and then merging them back down into develop. Among other things.
-
VaderNT16345y@wypaliz Of course they are. It took me way too long to understand one central thing about people:
People never do anything wrong. From their point of view, that is, because otherwise they wouldn't be doing it.
People react differently to being shown they're wrong. Some are thankful for the opportunity to improve themselves. Others get annoyed or, like that dev, throw a hissy fit. -
@VaderNT I like if someone tells me I did something wrong, especially coming from another dev. We are humans, so we might make mistakes - and those mistakes will eventually be made without being noticed by the one who makes it - which is why I like constructive criticism a lot in this area.
-
Got some criticism on the refactoring merge request I made today, because I missed some stuff.
Was glad that I got it.
Related Rants
-
iSwimInTheC2Wait what's that? You don't use version control on Production servers? You want me to do what? You want me ...
-
phuzzy5My code is reviewed twice, once by the team leader, and once by a tech lead. One told me to move a const from...
-
hax0r2The day you finally realize that you're not crazy because everyone on devRant experiences the same problems yo...
Working with a new team and I don't understand how this is normal or ok.
Me: Does anyone need help troubleshooting the broken build or can I revert this change that broke it so I can push my change out?
Dev: Stop build shaming me, I wouldn't leave the build in a failing state.
Me: Well, I wasn't sure how long to wait, before asking.... it's been broken for 4 hours.
Dev: It's the development environment, you should expect development to be going on.
Me: Yes and appears that this project architecture doesn't support any sort of isolation for development. So nobody can deploy anything except through the development branch.
Dev: That's what development is for IMO, so it doesn't bother me.
rant
build shaming
broken build
this is fine
totally normal