Please open a new issue or new pull request for bugs, feedback, or new features you would like to see. If there is an issue you would like to work on, please leave a comment and we will be happy to assist. New contributions and contributors are very welcome!
See the instructions below for how to contribute to this repository using a forking workflow.
-
Create a personal fork of the
poppy
repository by visiting its location on GitHub and clicking theFork
button. This will create a copy of thepoppy
repository under your personal GitHub account (hereby referred to as "personal fork"). Note that this only has to be done once. -
Make a local copy of your personal fork by cloning the repository (e.g.
git clone https://github.com/username/poppy.git
, found by clicking the green "clone or download" button.). Note that, unless you explicitly delete your clone of the fork, this only has to be done once. -
Ensure that the personal fork is pointing to the
upstream
poppy
repository withgit remote add upstream https://github.com/spacetelescope/poppy.git
(or use the SSH version if you have your SSH keys set up). Note that, unless you explicitly change the remote location of the repository, this only has to be done once. -
Create a branch off of the
develop
branch on the personal clone to develop software changes on. Branch names should be short but descriptive (e.g.new-database-table
orfix-ingest-algorithm
), and not too generic (e.g.bug-fix
). Consistent use of hyphens is encouraged.git branch <branchname>
git checkout <branchname>
- you can use this command to switch back and forth between existing branches.- Perform local software changes using the nominal
git add
/git commit -m
cycle:git status
- allows you to see which files have changed.git add <new or changed files you want to commit>
git commit -m 'Explanation of changes you've done with these files'
-
Push the branch to the GitHub repository for the personal fork with
git push origin <branchname>
. -
In the
poppy
repository, create a pull request for the recently pushed branch. You will want to set the base fork pointing topoppy:develop
and thehead
fork pointing to the branch on your personal fork (i.e.username:branchname
). Note that if the branch is still under development, you can use the GitHub "Draft" feature (under the "Reviewers" section) to tag the pull request as a draft. Not until the "Ready for review" button at the bottom of the pull request is explicitly pushed is the pull request 'mergeable'. -
Assign the pull request a reviewer, selecting a maintainer of the
poppy
repository. They will review your pull request and either accept the request and merge, or ask for additional changes. -
Iterate with your reviewer(s) on additional changes if necessary, addressing any comments on your pull request. If changes are required, you may end up iterating over steps 4.iii and 5 several times while working with your reviewer.
-
Once the pull request has been accepted and merged, you can delete your local branch with
git branch -d <branchname>
.
If you wish to, you can keep a personal fork up-to-date with the poppy
repository by fetching and rebasing with the upstream
remote. Do this for both the master
branch (shown below) and the develop
branch:
git checkout master
git fetch upstream master
git rebase upstream/master
Users can contribute to another user's personal fork by adding a remote
that points to their fork and using the nominal forking workflow, e.g.:
git remote add <username> <remote URL>
git fetch <username>
git checkout -b <branchname> <username>/<branchname>
- Make some changes (i.e.
add/commit
cycle) git push <username> <branchname>