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

[TASK]: Agree on and potentially document a process for recording outcomes of off-GitHub discussions #292

Open
matt-graham opened this issue Jan 24, 2024 · 2 comments
Assignees
Labels
task A task to be completed with the repo

Comments

@matt-graham
Copy link
Collaborator

matt-graham commented Jan 24, 2024

What needs to be done?

Following on from discussion in thread of #288. We sometimes have conversations about this project off-GitHub - for example during the semi-regular in-person hackathon events we've been running or on the internal ARC Slack workspace. To make sure those not present in these discussions are aware of the points raised and the rationale behind any decisions made, I would suggest we should have some minimal process around ensuring we sufficiently document such conversations to allow everyone the chance to contribute.

We generally already create issue corresponding to any decision about changes to be made, but sometimes these have been very barebones and not had any summary explaining why a decision was made. I would suggest we just ensure that in any such issue we give a (potentially very brief) summary of the points made in the initial issue comment. We could potentially even have an issue template specifically for this, though not entirely sure that makes sense?

@matt-graham matt-graham added the task A task to be completed with the repo label Jan 24, 2024
@samcunliffe
Copy link
Member

I acknowledge that #254 could have done with more text in the leading comment.

But FWIW I'm also OK with the workflow that we ended up having. It seems sane to me...

  • Potentially contentious changes were explicit in the name of the issue and PR,
  • comment challenging the decision on the PR was raised and PR marked as "changes requested" because changes were requested,
  • ...discussion ensued.

Modulo that I disagree with the comment here. In-person decisions are not final. Differing opinions are welcome. This is now explicitly codified in line 21 of our CoC.

@dstansby
Copy link
Member

I would suggest we just ensure that in any such issue we give a (potentially very brief) summary of the points made in the initial issue comment.

I'm 👍 this - it makes it much more inclusive as a project if everyone can see why decisions have been taken, and I don't think it adds much more effort on top of in person discussions to jot reasons behind a decision down. Indeed, it also helps remember why a decision has been taken later - I think someone who was in person commented on a thread somewhere that they had forgotten why a decision had been made!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
task A task to be completed with the repo
Projects
None yet
Development

No branches or pull requests

4 participants