"Can you review this pull request?"

Ok, sure

- Description in broken English
- HTML/CSS changes seemingly just for the fuck of it
- No user story listed OR
- User story listed has no description
- Mockup does not specify what should be changed
- Owner is offline because this entire team operates out of India
- Requirements said to exist but their location is unknown

Fail the pr for no context

Sounds like an easy PR.
Denied.

@SortOfTested it's every PR. The entire project is just offshore vendors. It's a total shit show.

@Root I requested changes and said to make sure to include test data / steps in the PR, but every PR is like this. Not only that, the scrum master / BA doesn't include adequate information in the user story and the business folk seem to be out of the loop on the projects current happenings.

I've had to rewrite code twice just because the description was too vague and ***after it was reviewed, merged, deployed and tested*** the business folk said "that's not right"

@molaram this project is already way behind schedule. I'm actually just helping out temporarily. Management knows how bad it is, at this point they're just trying to get shit finished.

Unfortunately I've been made authority on it. There's someone over me, but they have higher priority tasks and can only occasionally step in.

It's a dumpster fire.

My condolences.

@molaram I'm not a manager lol I don't have authority to do that. Plus the overhead on that may not be worth. There are people leaving the team, and I think one may be replaced, but the application is like 80% built at this point and like 200% past deadline.

Haha many times there were MR like this , sometimes I do leave a comment. And like other times I simply deny.