First pass on official tandem meetings

pirog commented 4 years ago

I just submitted code and now I want to make sure it's ready for primetime!!!

In order to do that I need to first go through the below checklist entitled "For Me" because I respect my coworkers and want to get this PR into the best place possible before i ask for their time. When i am done i will then summon reviewers for feedback and comment. These reviewers will have about a week to complete their review.

If they do not review in the time alloted or have no change requests or blocking concerns the person who opened the pull request will apply the "Ready to merge" label which will ping the Keepers of Faith to press the merge button.

For me

Bare minimum self-checks

What do you think of a person who only does the bare minimum?

[ ] I've read the Contributing Code Guide
[ ] I've updated this PR with the latest code from master
[ ] I've made sure my code fits into the right place
[ ] I've done a cursory QA pass of my code locally
[ ] I've written an acceptance test below for the reviewer
[ ] I've ensured all automated tests pass
[ ] I've connected this PR to an issue
[ ] I've gone into that issue and if it is a task with a linked "next task" i've gone into that task and made its next task or prompted a discussion there about what its next task should be

Pieces of flare

[ ] I've written a unit or functional test for my code
[ ] I've updated this repo's README if my code changes it
[ ] I've updated relevant documentation it my code changes it

Finally

[ ] I've requested a review with relevant people
[ ] I've moved this ticket into the "Ready for review" swimlane

For the reviewer

Acceptance Test

THE PERSON WHO SUBMITS THE PR SHOULD PUT THEIR ACCEPTANCE TEST HERE.

The acceptance test should detail the steps the reviewer can take to

Replicate the "issue" on the mainline site
Showcase the "fix" on the PR environment spun up on platform.sh.

Here is a very simple example you can follow

Steps to replicate

Provide a link to the prod or dev site
Navigate to the homepage, see that the background is neon green

Fix

Provide a link to the QA environment
Navigate to the homepage and see that the background is blue

Reviewer checks

I've completed the following tasks and requested changes from the submitter if changes were needed before a :+1: could be given.

[ ] I've read the QAing Code Guide
[ ] I've verified the developers QA checklist eg "For me" has been completed to a level i am ok with
[ ] I've manually reviewed the code and it checks out
[ ] I've manually validated the acceptance test(s)
[ ] I've summoned an additional reviewer if I need a second (or third, or fourth, ... or nth) opinion

For the Keeper(s) of the Faith

[ ] We've attempted to find a follow up issue if needed and if needed we've requested it be created
[ ] We've verified this does not wildly diverge from our mission, goals, roadmap, values etc

pirog commented 4 years ago

@dustinleblanc @mikemilano i've tried to codify the updated format/goals for our standups and eow meeting. i will be merging this in friday but wanted to give both of you an opportunity to comment beforehand if there are any additions/suggestions you have.

Here are the relevant pages in the PR environment https://pr-126-fal46ma-txmhst4apts56.us-2.platformsh.site/guides/daily-standups.html https://pr-126-fal46ma-txmhst4apts56.us-2.platformsh.site/guides/eow-meeting.html

Worth pointing out that these are intentionally "high level" eg they try not to "hardcode" implementation details that very well could change in the future eg the times of said standups. I think that makes the most sense but certainly open to being wrong.

@pirog: once this is merged in make sure you update the google calendar invites to point to the relevant docs @dustinleblanc: once this is merged in it might be worth just surfacing it in slack or a standup so everyone can get eyes on it

pirog commented 4 years ago

@mikemilano @dustinleblanc im going to merge this in but still free to comment or open a new issue/pr if you think it requires changes

thinktandem / tandem