nimblehq / kmm-templates

MIT License
2 stars 0 forks source link

[#7] [iOS] Setup CD: Firebase #49

Closed phongvhd93 closed 7 months ago

phongvhd93 commented 8 months ago

What happened 👀

Created 2 job for distribute the app to Firebase (Staging) and TestFlight (Production)

Insight 📝

Add jobs to two workflows for deploy iOS app to Firebase (staging) and TestFlight (production)

Proof Of Work 📹

Deploy Staging Build Screenshot 2023-12-18 at 09 23 56

Deploy Production Build

Screenshot 2023-12-18 at 09 20 39

github-actions[bot] commented 8 months ago
9 Warnings
:warning: Big PR
:warning: Uh oh! Your project is under 80% coverage!
:warning: android/src/main/java/co/nimblehq/kmm/template/ui/screens/home/HomeScreen.kt#L66 - This expression contains a magic number. Consider defining it to a well named constant.
:warning: shared/src/androidMain/kotlin/co/nimblehq/kmm/template/Platform.android.kt#L3 - The file name 'Platform.android' does not match the name of the single top-level declaration 'AndroidPlatform'.
:warning: shared/src/androidUnitTest/kotlin/co/nimblehq/kmm/template/androidTest.kt#L6 - The file name 'androidTest' does not match the name of the single top-level declaration 'AndroidGreetingTest'.
:warning: shared/src/commonTest/kotlin/co/nimblehq/kmm/template/commonTest.kt#L6 - The file name 'commonTest' does not match the name of the single top-level declaration 'CommonGreetingTest'.
:warning: shared/src/commonTest/kotlin/co/nimblehq/kmm/template/extensions/ResponseMappingTest.kt#L18 - Exception is a too generic Exception. Prefer throwing specific exceptions that indicate a specific error case.
:warning: shared/src/iosMain/kotlin/co/nimblehq/kmm/template/Platform.ios.kt#L5 - The file name 'Platform.ios' does not match the name of the single top-level declaration 'IOSPlatform'.
:warning: shared/src/iosTest/kotlin/co/nimblehq/kmm/template/iosTest.kt#L6 - The file name 'iosTest' does not match the name of the single top-level declaration 'IosGreetingTest'.

Kover report:

🧛 Unit Tests Code Coverage: 59.92%

Coverage of Modified Files:

File Coverage

Modified Files Not Found In Coverage Report:

Gemfile.lock Podfile.lock deploy_production.yml deploy_production.yml deploy_staging.yml deploy_staging.yml project.pbxproj

Codebase cunningly covered by count Shroud 🧛

Generated by :no_entry_sign: Danger

github-actions[bot] commented 8 months ago
1 Warning
:warning: This pull request is quite big (683 lines changed), please consider splitting it into multiple pull requests.

Current coverage for sample is 18.25%

No files affecting coverage found


Powered by xcov

Generated by :no_entry_sign: Danger

luongvo commented 7 months ago

@phongvhd93 please request reviews & complete this task soon 🙏