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

Upgrade sdk libraries to latest versions #2825

Merged
merged 138 commits into from
Feb 7, 2024
Merged

Conversation

ellykits
Copy link
Collaborator

@ellykits ellykits commented Oct 18, 2023

IMPORTANT: Where possible all PRs must be linked to a Github issue

Fixes #2816

Engineer Checklist

  • I have written Unit tests for any new feature(s) and edge cases for bug fixes
  • I have added any strings visible on UI components to the strings.xml file
  • I have updated the CHANGELOG.md file for any notable changes to the codebase
  • I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
  • I have built and run the FHIRCore app to verify my change fixes the issue and/or does not break the app
  • I have checked that this PR does NOT introduce breaking changes that require an update to Content and/or Configs? If it does add a sample here or a link to exactly what changes need to be made to the content.

Code Reviewer Checklist

  • I have verified Unit tests have been written for any new feature(s) and edge cases
  • I have verified any strings visible on UI components are in the strings.xml file
  • I have verifed the CHANGELOG.md file has any notable changes to the codebase
  • I have verified the solution has been implemented in a configurable and generic way for reuseable components
  • I have built and run the FHIRCore app to verify the change fixes the issue and/or does not break the app

ellykits and others added 5 commits October 16, 2023 18:18
@ellykits ellykits added Blocked Item is blocked by another prerequisite DNM DO NOT MERGE Work In Progress Describes an items that is not complete. Mostly used for scoping issues of discussions labels Oct 18, 2023
@ellykits ellykits self-assigned this Oct 18, 2023
Signed-off-by: Elly Kitoto <junkmailstoelly@gmail.com>
ellykits and others added 13 commits October 24, 2023 11:51
Signed-off-by: Elly Kitoto <junkmailstoelly@gmail.com>
Signed-off-by: Elly Kitoto <junkmailstoelly@gmail.com>
Signed-off-by: Elly Kitoto <junkmailstoelly@gmail.com>
Signed-off-by: Elly Kitoto <junkmailstoelly@gmail.com>
* Remove extra libraries | upgrade sdk workflow libs

* Update Kotlin version to 1.8.20

Signed-off-by: Elly Kitoto <junkmailstoelly@gmail.com>

---------

Signed-off-by: Elly Kitoto <junkmailstoelly@gmail.com>
Co-authored-by: Elly Kitoto <junkmailstoelly@gmail.com>
…nto upgrade-sdk-libraries

Signed-off-by: Elly Kitoto <junkmailstoelly@gmail.com>
Signed-off-by: Elly Kitoto <junkmailstoelly@gmail.com>
Signed-off-by: Elly Kitoto <junkmailstoelly@gmail.com>
Signed-off-by: Elly Kitoto <junkmailstoelly@gmail.com>
Issue resolved by using this property on the debug build variant
"enableUnitTestCoverage" instead of "isTestCoverageEnabled".
"isTestCoverageEnabled" is deprecated and uses jacoco under the hood.

Signed-off-by: Elly Kitoto <junkmailstoelly@gmail.com>
@ellykits ellykits removed Blocked Item is blocked by another prerequisite DNM DO NOT MERGE labels Nov 1, 2023
@ellykits ellykits marked this pull request as ready for review November 1, 2023 13:35
@ellykits
Copy link
Collaborator Author

ellykits commented Nov 1, 2023

The blocker has been resolved.

Signed-off-by: Elly Kitoto <junkmailstoelly@gmail.com>
Signed-off-by: Elly Kitoto <junkmailstoelly@gmail.com>
@ndegwamartin
Copy link
Contributor

@vitorpamplona I've reverted the dependencies workaround on a separate branch here : https://github.com/opensrp/fhircore/tree/no-dep-workaround

@vitorpamplona
Copy link
Contributor

Nice, are you getting the right versions on the HAPI Fhir now?

@pld pld enabled auto-merge (squash) February 6, 2024 18:44
@ndegwamartin
Copy link
Contributor

Nice, are you getting the right versions on the HAPI Fhir now?

Not yet, with this state it doesn't even compile. Created the branch so that it is easy for you to replicate the issue by just trying to build.

pld
pld previously approved these changes Feb 6, 2024
@ndegwamartin
Copy link
Contributor

Nice, are you getting the right versions on the HAPI Fhir now?

Not yet, with this state it doesn't even compile. Created the branch so that it is easy for you to replicate the issue by just trying to build.

@vitorpamplona were you able to replicate?

@ndegwamartin ndegwamartin enabled auto-merge (squash) February 7, 2024 15:44
@ndegwamartin ndegwamartin merged commit f38ada8 into main Feb 7, 2024
4 of 6 checks passed
@ndegwamartin ndegwamartin deleted the upgrade-sdk-libraries branch February 7, 2024 15:51
AbdulWahabMemon added a commit that referenced this pull request Feb 9, 2024
* main:
  Add icons to be displayed on eusm service point register. (#3047)
  Upgrade sdk libraries to latest versions (#2825)
  Make Login Screen Logo Size Configurable (#3045)

# Conflicts:
#	android/gradle/libs.versions.toml
#	android/quest/src/main/java/org/smartregister/fhircore/quest/ui/questionnaire/QuestionnaireActivity.kt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Work In Progress Describes an items that is not complete. Mostly used for scoping issues of discussions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate FHIR Core to the latest FHIR SDK artifacts