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

Contributing guide instructions for allowing commits from maintainers #917

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

dbeatty10
Copy link
Contributor

resolves #913

Problem

We've had cases where "Allow edits from maintainers" is not enabled by the contributor. This means that a maintainer can't push commits and all commits need to be initiated by the contributor instead. This lowers the collaboration possibilities and can ultimately necessitate closing the original PR and opening a new one.

Solution

Provide a link within the contributing guide within the "external contributors" section that provides instructions for enabling commits from maintainers.

Checklist

  • This code is associated with an issue which has been triaged and accepted for development.
  • I have read the contributing guide and understand what's expected of me
  • [N/A] I have run this code in development and it appears to resolve the stated issue
  • [N/A] This PR includes tests, or tests are not required/relevant for this PR
  • [N/A] I have updated the README.md (if applicable)

@dbeatty10 dbeatty10 marked this pull request as ready for review June 5, 2024 12:02
@dbeatty10 dbeatty10 merged commit 9e2b498 into main Jun 5, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Explain "Allow edits from maintainers" within the contributing guide
1 participant