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 consulted and informed #63

Merged
merged 3 commits into from
May 24, 2022
Merged

Add consulted and informed #63

merged 3 commits into from
May 24, 2022

Conversation

koppor
Copy link
Member

@koppor koppor commented May 20, 2022

Fixes #62

@cristiklein On the one hand, I thought of adding "responsible" and "accountable", however this felt to be too much fields (even though "deciders" says nothing about these two "roles". Nevertheless, for a "lean" template, it is OK. -- Should we add an ADR on that?

@cristiklein
Copy link
Collaborator

Nice PR @koppor !

People tend to understand "consulted" and "informed" really well. The comments you added in the template make these two roles even clearer.

However, in my experience, people get confused about who is "accountable" and who is "responsible". Furthermore, if decisions are mostly taken by consensus in small committees, then there might not be an "accountable" person.

Hence, I very much like your PR as it is right now.

I'm split on whether your comment and my comment deserves an ADR. Please decide and I'm happy to draft one.

Co-authored-by: Cristian Klein <[email protected]>
@koppor
Copy link
Member Author

koppor commented May 24, 2022

Thank you for sharing your thoughts! I was too quick in reading and started an ADR at https://github.com/adr/madr/blob/fix-62/docs/decisions/0015-include-consulting-informed-of-raci.md. I gave you write access to your repository for any improvements on that draft :).

@cristiklein
Copy link
Collaborator

@koppor I'm looking for the "jaw dropped" emoji. 😄

The ADR reads really well and is crystal clear. Nice work!

I'd say this PR is ready to merge from my side.

@koppor koppor merged commit 8e429c0 into main May 24, 2022
@koppor koppor deleted the fix-62 branch May 24, 2022 15:32
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.

Do we need "consulted" and "informed"?
2 participants