You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
During the 2021-09-29 call, @avigoldman asked about the process of contribution and who can do what. I think it's clear that we don't have well-defined processes/rules yet. @M-J-Robbins also pointed out that it could be useful to define some roles for the separate projects we work on.
I think the only thing we agreed on in this regard so far is to make it mandatory for another member to approve a PR before merging here.
Just thinking as I put up a PR, should we add some guidance on who to tag to review a PR.
I feel like tagging everyone is too spamming, but also don't want to exclude people.
Perhaps we need one or two people to be in charge of general house keeping*. Part of their role can be reviewing PRs. They can be the ones to be tagged as reviewers by default. They can then decide who's best to review and approve the PR (if not them) and tag them to review.
*We're sort of collectively trying to do this right now. Contribution here doesn't have to be in writing code or documentation. Keeping the whole group organised and keeping tabs on the various projects would be very helpful and valuable to us.
@dylanatsmith also shared some comments on Slack about GitHub code reviews that may be relevant here.
During the 2021-09-29 call, @avigoldman asked about the process of contribution and who can do what. I think it's clear that we don't have well-defined processes/rules yet. @M-J-Robbins also pointed out that it could be useful to define some roles for the separate projects we work on.
I think the only thing we agreed on in this regard so far is to make it mandatory for another member to approve a PR before merging here.
Common approaches in open source:
The text was updated successfully, but these errors were encountered: