search

nimblehq / jetpack-compose-crypto

This is an sample Crypto market prices app built with Jetpack Compose
https://nimblehq.co/
MIT License
21 stars 5 forks source link

[#2] Setup CICD pipelines #9

Closed hoangnguyen92dn closed 2 years ago

hoangnguyen92dn commented 2 years ago

https://github.com/nimblehq/jetpack-compose-crypto/issues/2

What happened 👀

Add the CICD pipelines to execute the following workflow:

Insight 📝

Proof Of Work 📹

Workflow success: Deploy staging to Firebase App Distribution

image

github-actions[bot] commented 2 years ago
7 Warnings
:warning: Big PR
:warning: /home/runner/.gradle/caches/transforms-2/files-2.1/8be03b37a8bfd2c061b0e810ebd34556/jetified-timber-4.7.1/jars/lint.jar#L0 - Lint found an issue registry (timber.lint.TimberIssueRegistry) which is older than the current API level; these checks may not work correctly.

Recompile the checks against the latest version. Custom check API version is 1 (3.1), current lint API level is 8 (4.1)
:warning: /home/runner/.gradle/caches/transforms-2/files-2.1/d98050e38158aafceef82a76ae97f036/jetified-annotation-experimental-1.0.0/jars/lint.jar#L0 - Lint found an issue registry (androidx.annotation.experimental.lint.ExperimentalIssueRegistry) which did not specify the Lint API version it was compiled with.

This means that the lint checks are likely not compatible.

If you are the author of this lint check, make your lint IssueRegistry class contain
  override val api: Int = com.android.tools.lint.detector.api.CURRENT_API
or from Java,
  @Override public int getApi() { return com.android.tools.lint.detector.api.ApiKt.CURRENT_API; }

If you are just using lint checks from a third party library you have no control over, you can disable these lint checks (if they misbehave) like this:

    android {
        lintOptions {
            disable "UnsafeExperimentalUsageError",
                    "UnsafeExperimentalUsageWarning"
        }
    }
:warning: app/src/debug/AndroidManifest.xml#L5 - On SDK version 23 and up, your app data will be automatically backed up and restored on app install. Consider adding the attribute android:fullBackupContent to specify an @xml resource which configures which files to backup. More info: https://developer.android.com/training/backup/autosyncapi.html
:warning: app/src/main/AndroidManifest.xml#L7 - On SDK version 23 and up, your app data will be automatically backed up and restored on app install. Consider adding the attribute android:fullBackupContent to specify an @xml resource which configures which files to backup. More info: https://developer.android.com/training/backup/autosyncapi.html
:warning: app/src/main/AndroidManifest.xml#L12 - Attribute networkSecurityConfig is only used in API level 24 and higher (current min is 23)
:warning: app/src/main/res/navigation/nav_graph_main.xml#L2 - The resource R.navigation.nav_graph_main appears to be unused

CoroutineTemplate Jacoco report:

Generated by :no_entry_sign: Danger

luongvo commented 2 years ago

@hoangnguyen92dn could you please invite us into firebase app?

hoangnguyen92dn commented 2 years ago

@luongvo @Wadeewee Added PoW 🚀 Please finalize your review 🙏