-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[FR]: Add android API 21 in CI #1490
Labels
enhancement
New feature or request
Comments
Jaehwa-Noh
added a commit
to Jaehwa-Noh/nowinandroid
that referenced
this issue
Jun 6, 2024
This was referenced Jun 6, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
ab78c44
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 #1485 (comment) .
If android 21 API was added in CI, we can find crash as early as possible, and can remove the threat.
Code of Conduct
The text was updated successfully, but these errors were encountered: