Contributing to CellLocator
There are many ways to contribute to CellLocator.
Submit a feature request or bug, or add to the discussion on the CellLocator issue tracker
Submit a Pull Request to improve CellLocator.
The PR Process, and Related Gotchas
How to submit a PR ?
If you are new to CellLocator development and you don’t have push access to the CellLocator repository, here are the steps:
Fork and clone the repository.
Create a branch.
Push the branch to your GitHub fork.
Create a Pull Request.
This corresponds to the Fork & Pull Model mentioned in the GitHub flow
guides.
If you have push access to this repository, you could simply push your branch
and create a Pull Request. This corresponds to the Shared Repository Model
and will facilitate other developers to checkout your topic without having to
configure a remote.
It will also simplify the workflow when you are co-developing a branch.
When submitting a PR, make sure to add a Cc: @cell-locator/developers comment to
notify CellLocator developers of your awesome contributions. Based on the
comments posted by the reviewers, you may have to revisit your patches.
How to integrate a PR ?
Getting your contributions integrated is relatively straightforward, here is the checklist:
All tests pass
Consensus is reached. This usually means that at least one reviewer added a
LGTMcomment and a reasonable amount of time passed without anyone objecting.LGTMis an acronym for Looks Good to Me.
Next, there are two scenarios:
You do NOT have push access: A CellLocator core developer will integrate your PR.
You have push access: Simply click on the “Merge pull request” button.
Then, click on the “Delete branch” button that appears afterward.