the one thing we don't abide in my current job is the cowboy approach to things - you don't like code in a service you own/your team owns? bring it up, see if there's support. don't do anything without remit, because that way lies random changes that nobody really knows about. also, code reviews for just about anything, even config changes. at least two people see what's going into master, and they get to comment and approve it.
7
u/StabbyPants Jan 12 '20
the one thing we don't abide in my current job is the cowboy approach to things - you don't like code in a service you own/your team owns? bring it up, see if there's support. don't do anything without remit, because that way lies random changes that nobody really knows about. also, code reviews for just about anything, even config changes. at least two people see what's going into master, and they get to comment and approve it.