Skip to content

Use adb install -g flag only for Android API above 23#2193

Merged
0xnm merged 1 commit into
developfrom
nogorodnikov/use-adb-install-g-flag-only-for-android-above-23
Aug 14, 2024
Merged

Use adb install -g flag only for Android API above 23#2193
0xnm merged 1 commit into
developfrom
nogorodnikov/use-adb-install-g-flag-only-for-android-above-23

Conversation

@0xnm

@0xnm 0xnm commented Aug 14, 2024

Copy link
Copy Markdown
Member

What does this PR do?

It doesn't work with Android below 23, giving Error: Unknown option: -g and resulting in the installation failure.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Base automatically changed from nogorodnikov/rum-5831/speed-up-generated-files-check to develop August 14, 2024 12:26
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.01%. Comparing base (881eb39) to head (ca575db).
Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2193      +/-   ##
===========================================
+ Coverage    69.92%   70.01%   +0.09%     
===========================================
  Files          726      726              
  Lines        26991    27005      +14     
  Branches      4526     4527       +1     
===========================================
+ Hits         18871    18906      +35     
+ Misses        6842     6829      -13     
+ Partials      1278     1270       -8     

see 37 files with indirect coverage changes

@0xnm
0xnm marked this pull request as ready for review August 14, 2024 12:53
@0xnm
0xnm requested review from a team as code owners August 14, 2024 12:53
@0xnm
0xnm merged commit 2529684 into develop Aug 14, 2024
@0xnm
0xnm deleted the nogorodnikov/use-adb-install-g-flag-only-for-android-above-23 branch August 14, 2024 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants