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-3903 #431

Closed
wants to merge 2 commits into from
Closed

TRUNK-3903 #431

wants to merge 2 commits into from

Conversation

k-joseph
Copy link
Member

[https://tickets.openmrs.org/browse/TRUNK-3903] added some javadocs and unit tests to my previous commits

@dkayiwa
Copy link
Member

dkayiwa commented Oct 30, 2013

Instead of simply putting the pull request comment as "TRUNK-3903", can you also add the ticket description?

@dkayiwa
Copy link
Member

dkayiwa commented Oct 30, 2013

By the way, do you also see the CI build failure with this commit?

@k-joseph
Copy link
Member Author

i have responded back on the ticket;
in summery, i have just run mvn clean install and it is a success, may be some one else can look into this and we can establish what is wrong and then way forward, thanks

@dkayiwa
Copy link
Member

dkayiwa commented Oct 31, 2013

It turns out to be the usual randomly failing scheduler test. You can just ignore it. :)

@k-joseph k-joseph closed this Dec 18, 2013
@k-joseph k-joseph deleted the TRUNK-3903 branch December 18, 2013 21:32
RandilaP pushed a commit to RandilaP/openmrs-core that referenced this pull request Jul 31, 2023
* (chore) Update CI status badge URL

* Fixup
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