tendomart

Results 46 comments of tendomart

Ooh thanks @joachim-junior and @bejanut for the reviews .Iam gonna look at the Travis logs and rectify the failure .

![Screenshot from 2021-04-23 18-18-23](https://user-images.githubusercontent.com/16301386/115894110-93008580-a461-11eb-838c-28bd7c5d2862.png) A Screenshot of the database for the new columns in **order_group** table i.e **care_setting** and **order_type_id**

Everything was cleaned up in the last [commit](https://github.com/openmrs/openmrs-core/pull/3647/commits/8a1abbf24b8d2d27d7e22d9a59ccb5255a018f3d) , Iam not really sure why this is still here ..

@abertnamanya it looks like some files are missing license headers

@brownbread269 is this attached to any ticket ? If yes you may want to include a link and reflect that in your title as well.

Did you run integration tests before commit? Because you have an issue at `/home/runner/work/openmrs-core/openmrs-core/api/src/test/java/org/openmrs/api/OrderServiceTest.java:[2617,21] illegal start of expression `

There is a problem at **/home/runner/work/openmrs-core/openmrs-core/api/src/main/java/org/openmrs/api/context/Context.java:[1441,25] unreachable statement** please run `mvn clean package` after making changes and before doing a commit to avoid these failures.

And pay attention to all the comments which I made on this PR. Changes still failing. did you fetch the latest changes before pushing ?

@abertnamanya hope you're looking into the failed tests