Wiki Spaces
Documentation
Projects
Resources
Get Help from Others
Q&A: Ask OpenMRS
Discussion: OpenMRS Talk
Real-Time: IRC Chat | Slack
Clone the fork to your local machine (replace "yourusername"):
git clone https://github.com/yourusername/openmrs-core.git
Go into the folder just created and set up the "upstream" remote so you can eventually pull changes from the main repository (see "Configure Remotes" on http://help.github.com/fork-a-repo):
git remote add upstream https://github.com/openmrs/openmrs-core.git
Fetch and track all branches on remote repositories.
git fetch --all
After you cloned the repository you can list available branches:
git branch -a
The currently checked out branch is highlighted with an asterisks. The master branch is the latest development branch (trunk was its counterpart in SVN).
Pull changes from the upstream remote:
git pull --rebase upstream master
The '--rebase' option reverts any of your commits which are not in upstream/master, then fast forwards your local branch to upstream/master and finally applies your commits on top of that. This allows you to avoid merge commits and keep the history linear, but must not be used if you want others to work with you on a branch in your fork.
Optional pro tip: To save some key hits we recommend to setup an alias for 'pull --rebase' by running
git config --global alias.up "pull --rebase"
From now on you will be able to use 'git up upstream master' instead of 'git pull --rebase upstream master'.
Push changes to your fork on github:
git push
This command pushes all your commits from all your tracked branches to your fork. If your fork has commits, which are not in your local repository, the branches containing these commits will not be pushed. You will need to pull on these branches first to have them pushed.
We do not advise you to work on master or maintenance branches of openmrs-core, but create topic branches instead. This way you will be able to send us pull requests to merge your code back to the main repository. See "Create Topic Branches" below, please take a look at our pull requests tips page.
See Maven wiki page for how to build, compile, etc.
This will check out the code for "openmrs-core" (the core OpenMRS application):
git clone https://github.com/openmrs/openmrs-core.git
Checkout out a new local branch based on your master and update it to the latest. The convention is to name the branch after the current ticket e.g. TRUNK-123.
Make sure you have committed any new files that are not under version control otherwise you will lose them when you run *git clean -df*
git checkout -b TRUNK-123 master git clean -df git pull --rebase upstream master
Push the branch to your fork. Treat it as a backup.
git push origin TRUNK-123
Pro tip! Git has tab completion here!
NOTE: If the ticket you are working on requires you to work off a development branch say 1.9.x, you will have to use the development branch name instead of master for the instructions in this section. E.g. instead of git pull --rebase upstream master you will run git pull --rebase upstream 1.9.x, where 1.9.x is the development branch you are coding off.
Add changes to a commit (stage them). To see what files have changed.
git status
To stage all changed and new files.
git add -A
To pick only some files.
git add -i
You will see a summary of changed and new files. You need to choose '2' to mark files as updated. Now you need to pick files, which you want to mark as updated. You can specify them as a list 1, 2, 3, as a range 1-3, or simply * to select all. Confirm by hitting the enter key twice. If there are new (untracked) files, choose '4' and pick files the same way. Choose '7' to quit.
Commit changes.
git commit -m "TRUNK-123: Put change summary here (can be a ticket title)"
Please remember to specify the current ticket id in your commit message.
Optionally run mvn install -DskipTests=true before your commits so that the formatting is fixed.
If you need to take a break from your work and continue later, you should backup your code by committing changes and pushing to your fork.
git push origin TRUNK-123
If you absolutely need to and you know what you are doing, add the '-f' option to force the push, which means overwriting history if necessary. You must NEVER use this option with the main repository (upstream: https://github.com/openmrs)!
Before you pick up your work again, update to the latest code.
git pull --rebase upstream master
If you have uncommitted changes and git refuses to merge, you can stash your changes, perform a pull and then unstash them.
git stash git pull --rebase upstream master git stash pop
If you have conflicts, you need to resolve them by editing conflicting files, adding them to index, committing and then pulling again. Alternately you could try with a different merge strategy.
git pull --rebase -X patience upstream master
If you want to nuke all your commits run 'git reset --hard upstream master'. If you want to nuke only last commit run 'git reset --hard HEAD~1'.
Update your branch to the latest code.
git pull --rebase upstream master
If you have made many commits, we ask you to squash them into atomic units of work. Most of tickets should have one commit only, especially bug fixes, which makes them easier to back port. To squash three commits into one, do the following:
git checkout master git pull --rebase upstream master git checkout TRUNK-123 git rebase -i HEAD~3
In the text editor that comes up, replace the words "pick" with "squash" or more preferably "f" next to the commits you want to squash into the commit before it. Save and close the editor, and git will combine the "squash"'ed commits with the one before it. Git will concatenate the commit messages if "pick" is replaced with "squash".
Helpful hint: You can always edit your last commit message, before pushing, by using:
git commit --amend
Make sure all unit tests still pass
mvn clean package
Push changes to your fork
git push -f
Helpful Hint: its always a good Practice , that before you push code to your remote repository , first pull all the latest changes from the remote branches ( upstream and origin).
This is important in such a way that , the time you spend working on your ticket, new changes are being pushed to the remote repositories by other people , hence this may lead to merge conflicts if you try to push the same changes already pushed by some one else.
pull all the latest changes from the upstream branch
git pull --rebase upstream master
if you are using EGit, you may need to synchronize the changes you made on your local copy against either your fork or your remote master copy (upstream).
To compare with upstream:
To compare your changes against your fork (origin),
After you requested a code review you need to wait for a full committer to look at your work and either merge your changes or ask for improvements. We are doing our best so that it does no take longer than a few days for you to get feedback.
Reviews are normally done in github as comments on your pull request (see https://github.com/openmrs/openmrs-core/pull/93 for example). You can see all comments on the discussion tab. If you did follow up commits, some comments may be hidden and you just need to click 'Show outdated diff' to see them. When a reviewer has finished commenting on the pull request, the ticket will be moved to the 'Rework Needed', 'Committed Code' or 'Closed' stage. If it is 'Committed Code' or 'Closed', your work has been merged, the pull request closed and your job is done.
In order to make improvements you just need to commit to the same branch you created for the pull request and push to your fork. Github will automatically detect new commits and add them to your pull request. After you have made all improvements you need to move the ticket back to the 'Request Review' stage. The reviewer will be automatically notified to look at your code again.
Sometimes it may take a few rounds of reviews for the code to be ready so do not be discouraged. It is our common effort to keep good quality of code.
After your pull request has been accepted, the ticket is closed, and you have no further need of the branch you created, you can clean things up by deleting it from your local repository and your fork:
git branch -D TRUNK-123 git push origin :TRUNK-123
(TODO, link to place in the "Creating Modules" hierarchy instead?)
The OpenMRS organization on GitHub (http://github.com/openmrs) can be used to host community OpenMRS projects. All bundled modules managed within git are kept under this organization with the naming convention "openmrs-module-<moduleid>". All community-supportes modules are welcome and encouraged to be kept under the OpenMRS organization as well.
Other organizations and authors can maintain OpenMRS modules under other accounts within github but are encouraged to use the same naming convention "openmrs-module-moduleid" naming convention.
git commit -m 'adding module to repository'
git remote add origin
https://github.com/openmrs/openmrs-module-appointment.git
git pull origin master
git push origin master
When using Git for your project, please use the following naming conventions:
openmrs-module-htmlformentry
)openmrs-distro-referenceapplication
)openmrs-contrib-atlas
).When creating a new module or contrib, please e-mail code@openmrs.org to request a module ID or contribution name, including your OpenMRS ID, a description of your module/contribution, and your proposed module ID or contribname. The conversation with the "code" e-mail group is used to avoid duplicates and maximize consistency of naming; you should expect a module ID to be decided within one day of your request. Eventually, we hope to create a more automated process for ensuring uniqueness of module identification & awareness of existing modules.
Github is currently the largest git hosting repository available. We want to join the community of developers there as well as eliminate another headache of providing our own code hosting on our servers.
5 Comments
Joaquin Blaya
I wanted to make changes to openmrs-core and found this to be easier. Don't know if it's completely right, but I was able to modify the code and submit a pull request to openmrs
Rafal Korytkowski
There are 2 problems.
1. You won't be able to pull changes (update your fork) from github.com/openmrs/openmrs-core if you don't add upstream.
2. If you send a pull request off your master branch, then you will not be able to send us another pull request from your master branch before the first one is closed. Moreover if you push more changes to your master branch before the pull request is closed, they will be added to the pull request, even though they may be not relevant to that pull request.
I will experiment with the egit plugin when I have some time to get a tutorial for those who do not like the command line :-)
Joaquin Blaya
Thanks Rafal. Good to know. This worked for me because I needed to make a single pull request, then I'll delete this (I'm not a full time programmer), but I definiltey see that constant committers to OpenMRS shouldn't do this.
Roger Friedman
Tip:
If you are using a code generator such as jaxb, make sure that the generated files are excluded from git. Maven will regenerate the files each time you build, causing deltas that can ping-pong between repositories in an annoying way. Three ways to exclude them:
Daniel Kayiwa
FWIW, if you need to clone just a single branch like master, you could also try something like:
git clone -b master --single-branch https://github.com/openmrs/openmrs-core.git