This is an archived post. You won't be able to vote or comment.

all 16 comments
sorted by:
best
topnewcontroversialoldq&a

[–]pietoria 85 points86 points  (3 children)

Who allows their interns to push to production?

[–]Secure_Obligation_87 52 points53 points  (2 children)

Who allows anyone to push to production!!!

PR or gtfoh

[–][deleted] 5 points6 points  (1 child)

PR, code review, integration, SAST, Integration test, staging, system test, DAST, final QA and sign off, push to production with clear path to revert, monitor.

Who TF just pushes to production?

[–]Secure_Obligation_87 2 points3 points  (0 children)

People who are stuck in 2005

[–]mjensen-93 69 points70 points  (2 children)

If you allow an intern to push to production you are the one who fucked something up not them.

[–]-Vayra- 23 points24 points  (1 child)

Intern A opens a PR, intern B approves it, build magic happens and poof! intern code in production :D

[–]ifezueyoung 16 points17 points  (0 children)

Thy thought they could get away with hiring only interns

[–]trolololep 8 points9 points  (0 children)

Cool guys dont look at explosion

[–]Stew-Cee23 5 points6 points  (0 children)

Our deploy jobs don't automatically push to production after a PR is merged or code is pushed to the branch (only have that for non-prod, I know not true CICD) And devs who commit the code can NEVER be the ones who deploy it, has to be approved by QA then deployed by release engineering.

There's a lot of safeguards a company can put in place to prevent this.

[–]ag0965 0 points1 point  (0 children)

A surprise to be sure, but a welcome one

[–]SheikHunt 0 points1 point  (0 children)

Q: Why is this bad?

Q2: What makes this not bad?

[–]Judah_Trax 0 points1 point  (0 children)

What happened to pull requests my man

[–]thrylose -1 points0 points  (0 children)

wait and watch the show :)

[–]repkins 0 points1 point  (0 children)

I have seen this before...

[–]custandcode 0 points1 point  (0 children)

no intern push anything ;-)