Skip to content
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

Trunk 4109.1 #727

Closed
wants to merge 8 commits into from
Closed

Trunk 4109.1 #727

wants to merge 8 commits into from

Conversation

akshika47
Copy link
Contributor

No description provided.

@ghost
Copy link

ghost commented Mar 6, 2014

Please use more a descriptive pull request title, similar to that as described for commit messages in https://wiki.openmrs.org/display/docs/Using+Git under step 3 in section "Work on your task".

@akshika47
Copy link
Contributor Author

Yes, I will, Sorry about it, this commit was done a time before. Before
Daniel asked me to do so. I will do in that manner next time for sure.

On Thu, Mar 6, 2014 at 7:15 PM, Michael Downey notifications@github.comwrote:

You should use more descriptive commit comments, as described in
https://wiki.openmrs.org/display/docs/Using+Git under step 3 in section
"Work on your task".

Reply to this email directly or view it on GitHubhttps://github.com//pull/727#issuecomment-36888668
.

Akshika Wijesundara
Department of Computer Science & Engineering
University of Moratuwa
Sri Lanka,

@dkayiwa
Copy link
Member

dkayiwa commented Mar 7, 2014

Create a new pull request that does not have all these commits. And also remember to include some text descriptions in your commits and pull requests.

@dkayiwa dkayiwa closed this Mar 7, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants