ProgrammerHumor
damnItsTime
Discussion
If that's anything other than "used autoformatter to fix whitespace" or "optimised SVG files", then rejected for being too large
Or, maybe in some cases, auto-generated definition files or some such (although often that'd make more sense to be part of build than checked in)
tip: for formatting prs, use .git-blame-ignore-revs so git blame does not blame whoever did that formatting commit, but whoever last changed the actual code
Thanks! Git blame gets used a LOT at my job, so this'll come in handy.
That's one hell of a tip. I didn't know about this feature, thanks a lot!
At my job we just refer to an empty folder called "doc"
Huh, I wonder if Perforce has something like this.
Testdata for unit tests has been the cause of my biggest PRs.
Nice guess, you are correct!
Ooh, on which one? Autoformatter?
At least that should be simple enough to review - run the autoformatter yourself (because they also checked in autoformatter config, right?) and see if it's the same
Yeah about that... We are not yet using autoformatters. I tried to explain it to my boss, but he did not want me to spend time setting it up. We do have a readme in which all conventions are mentioned...
Rookie mistake, you don't ask about shit like that.
I didn't. I got fed up with my boss' feedback on my code. (Biggest one was not being allowed to use a switch statement.) So, naturally, I had a list of all remarks that I found unlogical so my code wouldn't be rejected. He found out about the list, got real mad at me for a day. Then, he introduced a "code book" in which are basically naming conventions and limits on what features to use.
I'm cleaning up my resume tonight, the final straw was not being promoted after a year. I make 2200 a month gross based in the netherlands. (32 hours)
oh man.. I'm over like "wtf is a seitch statement", thinking it must be something really cool I've never heard of.. embarrassing
My bad, I meant switch
yeahh, I know.. and it should have been obvious.. but I am still enjoying the first sips of my morning coffee and needed google to hit me with "did you mean switch statement" first
What about my 65 thousand lines of code though.
On our company we do format check at pipeline level so it fails. You are forced to always format it so it is always up to date
My autoformatter of choice is the find/replace in notepad++
Find four spaces, replace with \t
Not the same if 13000 lines of C or an updated lock file… Context matters.
It is a lot of whitespace with some added features sprinkled in between.
Reject that.
Whitespace changes and added features should be separate commits. They can be in the same PR, but not the same commit.
(Unless use squash commits. Then they need to be separate PRs too.)
At least someone wants to contribute haha no, good luck with that
Just decline it and tell them to break it down into more granular PRs
People have really screwed up the grammar of this meme. It should be either "It is my great disappointment to inform you" OR "It is with great disappointment that I inform you", not a mashup of the two.
Don't look at the singular gentleman.
Ok I will review asap after my feature is done, that’s really soon (the next milestone)
The initial developper: « It is just adding plurals in the wording »
Edit: Typo
This is very simple: the PR is too big. Please split changes into smaller PRs
I'm preparing one MR to send before I take my vacation.
Right now it's +13000 -15000
Well, at least they were kind enough to inform you that it is a trap beforehand.
Though, does that make it not a trap...?
Do two or three random and generic comments, then approve and hope someone else has to deal with the consequences
When you realize your career now includes babysitting code, not just writing it.
Also, playing policeman and forcing people to enforce guidelines and the likes, until one stubborn junior thinks he too good for it and find yourself triggering a PIP after repeated offence
What's a PIP?
Performance Improvement Plan. In theory, it's an agreement to correct how you're performing your job. If you don't shape up, you're fired. In reality, it's usually just a CYA by the employer to say "well, we tried" before firing an underperforming employee.
A process you have imposed on you by your supervisor because of bad performance, often resulting in a termination of contract (it wasn't always like this, but today it is a bad omen)
Thanks
Short for performance improvement plan, if I'm correct. Ironic, if so, as it seems to be interpreted as a step before termination by many
Sure thing
Performance Improvement Plan, sugar coat and a severance waiver so that employer can sack you without any reason, deliverables are often unrealistic to begin with
It's a death sentence, once you get one bail and go to the next job and survive until you get one there
I mean, I knew this was goin to happen at some point. Just expressing my disappointment.
Bruh, who the hell submitted a school essay as PR
One of the hardest things to find out. Your company thinks your more efficient doing reviews than coding. The worst part is they are probably right.