Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore(ci): check e2e main commits against Android API level 27 #6031

Merged
merged 136 commits into from
Dec 12, 2024

Conversation

bakoushin
Copy link
Contributor

@bakoushin bakoushin commented Sep 18, 2024

Description

We want CI to check the app against the highest and lowest supported Android API levels. The highest supported API level is being checked in the PR CI. The lowest level should be checked after merging the PR into a main branch.

Currently, the lowest API level we support is 24, but the levels 24, 25, and 26 are failing the CI for some reason.

Setting Android API level 27 as the lowest one for now.

  • 24 is failing due to Let's Encrypt root certificate expiration
  • 25 is failing (RET-1274 to investigate)
  • 26 is failing (RET-1275 to investigate)

Test plan

CI succeeds, like in this workflow: 76f5859

Related issues

Backwards compatibility

NA

Network scalability

NA

Copy link

codecov bot commented Sep 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.92%. Comparing base (33558ee) to head (5f7e643).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##             main    #6031    +/-   ##
========================================
  Coverage   88.92%   88.92%            
========================================
  Files         735      735            
  Lines       31366    31366            
  Branches     5821     5515   -306     
========================================
  Hits        27892    27892            
  Misses       3429     3429            
  Partials       45       45            

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 33558ee...5f7e643. Read the comment docs.

@bakoushin bakoushin changed the title chore(ci): check e2e main commits against Android API level 24 chore(ci): check e2e main commits against Android API level 27 Dec 11, 2024
@bakoushin bakoushin marked this pull request as ready for review December 11, 2024 14:12
Copy link
Collaborator

@MuckT MuckT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this! Using Android 8 as the baseline makes sense as the vast majority of Android Users are on newer versions: Mixpanel.

Copy link
Collaborator

@kathaypacific kathaypacific left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥳

@bakoushin bakoushin enabled auto-merge December 12, 2024 10:25
@bakoushin bakoushin added this pull request to the merge queue Dec 12, 2024
Merged via the queue into main with commit b40e2ef Dec 12, 2024
15 checks passed
@bakoushin bakoushin deleted the alex/e2e-main-android-24 branch December 12, 2024 11:06
github-merge-queue bot pushed a commit that referenced this pull request Dec 16, 2024
### Description

Follow up to #6031 

The Android API level must be specified not as a string but as a number,
as is expected in
[e2e-android.yml](https://github.com/valora-inc/wallet/blob/c6e23572ea8db100a3de9d65f0bae6d3f843b4c3/.github/workflows/e2e-android.yml#L7)

### Test plan

CI works

### Related issues

- Related to RET-1194

### Backwards compatibility

NA

### Network scalability

NA
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.