Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add offboarding checklists #923

Closed
wants to merge 14 commits into from
Closed

Add offboarding checklists #923

wants to merge 14 commits into from

Conversation

mercadon
Copy link
Contributor

No description provided.

@mercadon mercadon requested a review from sqs as a code owner May 11, 2020 20:47
@mercadon mercadon marked this pull request as draft May 11, 2020 20:53
@mercadon mercadon requested a review from nicksnyder May 12, 2020 21:26
@mercadon mercadon marked this pull request as ready for review May 12, 2020 21:26
Add engineering offboarding.
@nicksnyder
Copy link
Contributor

I am worried about these offboarding checklists getting out of date and desynchronized with the onboarding checklists.

Is there anything on the offboarding checklist that doesn't map to an item in an onboarding checklist?

If not, I propose that we just maintain an onboarding checklist and the offboarding process is to execute the onboarding checklist except take the opposite action.

This would be easier if we rephrased the list items. For example: "on call rotation" not "add to oncall rotation", "recurring 1-1" not "create a recurring 1-1".

Then these list items can be copied into an issue/doc titled onboarding or offboarding and it is obvious what action to do.

@mercadon
Copy link
Contributor Author

mercadon commented May 14, 2020

Is there anything on the offboarding checklist that doesn't map to an item in an onboarding checklist?

There are ~3 things that don't map to the onboarding checklist, and there are ~8 things from that onboarding list that don't map to the offboarding checklist. I hear you though– it is an extra list to keep updated and will require cross-referencing these two checklists, but I think there are enough differences that if we didn't have separate docs some items/actions would be unclear.

For now, I think it'll be valuable to have both an onboarding and offboarding checklist– if it does become too much of a burden to keep them both updated I'm happy to revisit combining them if we can make it clear what belongs to onboarding only vs onboarding+offboarding.

@sqs
Copy link
Member

sqs commented Aug 5, 2020

@mercadon Do you still intend to merge this PR? If not, please close it. I'll close it in a couple days if I haven't heard back (I'm just trying to clean up our PR list).

@sqs sqs changed the base branch from master to main September 5, 2020 04:38
@mercadon mercadon closed this Apr 7, 2021
@nicksnyder nicksnyder deleted the offboarding-1 branch July 23, 2021 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants