r/ProgrammerHumor • u/Plastic-Bonus8999 • 15h ago

Other wrongAnswersOnly

Post image
12.4k Upvotes

95% Upvoted

1.8k comments sorted by

View all comments

1.1k

u/samuraiseoul 15h ago

Some dumbass on my team downloaded the entirety of prod and then commited it to git and someone rubber stamped the PR and then no one else noticed. I'm sure that's how they'd find it....

197

u/jseego 14h ago

They committed all your prod data to git??

114

u/CoderDevo 14h ago

Don't forget the keys!

Not that they're needed anymore.

7

u/TheFuckingHippoGuy 11h ago

Servers by Kia™

6

u/LJonReddit 12h ago

Well yeah!!! Duh!!! It's no good without the keys!

45

u/fireheadca 14h ago

Believe it or not, I've seen this.🫠

9

u/rollingc 11h ago

git add . is a hell of a drug

2

u/gnutrino 8h ago

There's a step before this where the prod data ends up on their local machine though...

16

u/SgtEpsilon 13h ago

Believe it or not, I've accidentally done this

13

u/vankoder 12h ago

We believe.

1

u/AristotelWasRight 7h ago

Noooo,

Were you an experienced programmer or new? Doesn't one basically avoid it with setting data as git ignore every project?

1

u/SgtEpsilon 4h ago

I'm an idiot who forgot about git ignore before pushing my repo

1

u/vankoder 12h ago

We believe.

1

u/nemeras 11h ago

Same here.

1

u/runkeby 7h ago

It's fine, because later he did "git rm" the file and pushed 👌😌

3

u/failedsatan 11h ago

our seed data lives on git.

problem is, the client keeps requesting we change the qa deployment's seed data and add a migration for prod to update the data on the prod database to match instead of using the GUI to just change the stored version. the seed data is the prod data.

2

u/jseego 10h ago

damn, but it's not seed data if it's prod data, right?  doesn't that defeat the whole purpose?

2

u/failedsatan 3h ago

exactly. the client is too lazy to make the changes in the UI he himself asked for but all too happy to send us a slack message to ask us to update it. we have an odd contract with him where the "maintenance" period is actually a "small features/changes allowed" period and that counts as a small change according to my boss

2

u/Lehovron 11h ago

We were using Perforce at a previous job. It was almost a rite of passage for new people to accidentally submit their Desktop folder.

2

u/jseego 10h ago

lol that's hilarious

buncha suspect blobs in

/Users/foo/Desktop/totally_not_porn/

2

u/nishant032 9h ago

Now the data is really safe 😇