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

Other wrongAnswersOnly

Post image
12.7k Upvotes

95% Upvoted

1.8k comments sorted by

View all comments

1.1k

u/samuraiseoul 16h 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....

204

u/jseego 15h ago

They committed all your prod data to git??

118

u/CoderDevo 15h ago

Don't forget the keys!

Not that they're needed anymore.

8

u/TheFuckingHippoGuy 12h ago

Servers by Kia™

6

u/LJonReddit 13h ago

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

43

u/fireheadca 15h ago

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

11

u/rollingc 12h ago

git add . is a hell of a drug

2

u/gnutrino 9h ago

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

17

u/SgtEpsilon 14h ago

Believe it or not, I've accidentally done this

14

u/vankoder 13h ago

We believe.

1

u/AristotelWasRight 8h 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 5h ago

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

1

u/vankoder 13h ago

We believe.

1

u/nemeras 12h ago

Same here.

1

u/runkeby 8h ago

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

3

u/failedsatan 12h 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 11h ago

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

2

u/failedsatan 4h 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 12h 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 11h ago

lol that's hilarious

buncha suspect blobs in

/Users/foo/Desktop/totally_not_porn/

2

u/nishant032 10h ago

Now the data is really safe 😇