Skip to content

Added section about closing formatting PRs without review #471

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

astrofrog
Copy link
Member

This is a draft, to try and address the concept brought up during the coordination meeting that we should be able to close unsolicited formatting/restructuring PRs that don't have a clear benefit. Happy to iterate on the wording!

Copy link
Member

@pllim pllim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good in principle. Thanks!

Should we also add a blurb about restructuring PR claiming to have performance boost must present benchmarking result that can be confirmed separately by other maintainers, or is that too much?

@@ -39,6 +39,16 @@ follows these guidelines.
After successful review, the maintainer(s) will formally approve the pull
request.

While we welcome pull requests for anyone, pull requests that are purely related
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
While we welcome pull requests for anyone, pull requests that are purely related
While we welcome pull requests from anyone, pull requests that are purely related

@pllim
Copy link
Member

pllim commented Jun 30, 2025

I think there was also a comment at the meeting that this document should be linked from astropy core dev docs?

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.

2 participants