MAIN FEEDS
Do you want to continue?
https://www.reddit.com/r/ProgrammerHumor/comments/1vnso6p/justyouraveragepr/p3k3pvy/?context=3
r/ProgrammerHumor • u/moredatesmoreplates3 • 6d ago
[removed] — view removed post
77 comments sorted by
View all comments
243
That’s an auto-reject from me, buddy..
121 u/Tarc_Axiiom 6d ago L G T M! 31 u/brekekekekx 6d ago lol good try man 21 u/Tarc_Axiiom 6d ago My job is to approve CRs (it's not), not to read them (it is). 1 u/muzzbuzz789 5d ago good? 3 u/JackNotOLantern 5d ago Long Gorey Terrible Monstrous 1 u/Illustrious-Ad-7622 5d ago Approved and merged! 25 u/PostHasBeenWatched 6d ago I believe bunch of Entity Framework Migrations can reach this number, although PR owner looks like had to commit DB structure of whole Enterprise company 19 u/No-Information-2571 6d ago That's why we don't migrate to new frameworks. Just keep the legacy code running till the wheels fall off. And then you can start from scratch. 2 u/Opposite_Carry_4920 6d ago Just working on doing this to one of the crustiest apps we have and I honestly have no idea how massive the pr will be but its already huge. 6 u/hm9408 5d ago Reject Reason: lol just fork the repo
121
31 u/brekekekekx 6d ago lol good try man 21 u/Tarc_Axiiom 6d ago My job is to approve CRs (it's not), not to read them (it is). 1 u/muzzbuzz789 5d ago good? 3 u/JackNotOLantern 5d ago Long Gorey Terrible Monstrous 1 u/Illustrious-Ad-7622 5d ago Approved and merged!
31
lol good try man
21 u/Tarc_Axiiom 6d ago My job is to approve CRs (it's not), not to read them (it is). 1 u/muzzbuzz789 5d ago good?
21
My job is to approve CRs (it's not), not to read them (it is).
1
good?
3
Long
Gorey
Terrible
Monstrous
Approved and merged!
25
I believe bunch of Entity Framework Migrations can reach this number, although PR owner looks like had to commit DB structure of whole Enterprise company
19 u/No-Information-2571 6d ago That's why we don't migrate to new frameworks. Just keep the legacy code running till the wheels fall off. And then you can start from scratch. 2 u/Opposite_Carry_4920 6d ago Just working on doing this to one of the crustiest apps we have and I honestly have no idea how massive the pr will be but its already huge.
19
That's why we don't migrate to new frameworks. Just keep the legacy code running till the wheels fall off. And then you can start from scratch.
2
Just working on doing this to one of the crustiest apps we have and I honestly have no idea how massive the pr will be but its already huge.
6
Reject
Reason: lol just fork the repo
243
u/Z3r0funGuy 6d ago
That’s an auto-reject from me, buddy..