Pull Request Check List¶
The pull request (commonly referred to as a PR) check list below is an outline of the steps that should be taken when making a contribution to a SunPy repository on Github.
- Review and test changes locally on your machine (see Testing Guidelines).
Double check that a pull request does not exist for the changes you are making. Ideally, check that there is an issue that details what you want to change and why.
If you are contributing code, review the Coding Standards page.
See the Developer’s Guide for guidelines regarding code tests, documentation, and other types of contributions.
Have you tested your changes with the latest version of
sunpy? If not, update your local copy from your remote repository on Github.
- Add a changelog to your pull request.
A changelog is a short record of the type of changes made in your pull request. Other users are the intended audience, and you can have multiple logs per pull request.
- Maintainers will review your pull request Pull Requests and GitHub Teams.
Tweak anything that others highlight and push the changes to your branch. You can also commit suggestions either in bulk or single commits via the Github user interface.
Discuss possible changes or improvements in the comments with the reviewers.
- Review the Continuous Integration (CI) What runs on our Continuous Integration tests and fix any errors or warnings that are found.
If you are confused by an error that the continuous integration is giving you, submit a comment in your pull request.
- Ask questions if you get stuck or confused at any point!
Open-source projects are about communication and collaboration.
This guide is partially based on Astropy’s Development Workflow.