android / nowinandroid

A fully functional Android app built entirely with Kotlin and Jetpack Compose
Apache License 2.0
15.92k stars 2.81k forks source link

[FR]: Add android API 21 in CI #1490

Open Jaehwa-Noh opened 4 weeks ago

Jaehwa-Noh commented 4 weeks ago

Is there an existing issue for this?

Describe the problem

Some code change not cover the lower android APIs and there's no way to check it on CI.

Describe the solution

Add API 21 at CI.

Additional context

In this commit, the android API 23 was removed from CI. https://github.com/android/nowinandroid/commit/ab78c44e791fd5755cfc7abf27638c2dac73e593 I don't know exactly the history and what happened at that time, but I felt the needs of cover lower API in CI from https://github.com/android/nowinandroid/pull/1485#issuecomment-2150505641 .

If android 21 API was added in CI, we can find crash as early as possible, and can remove the threat.

Code of Conduct