-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat(ramp): add webview debug by env vars #7341
Conversation
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7341 +/- ##
=======================================
Coverage 34.60% 34.60%
=======================================
Files 1017 1017
Lines 27144 27144
Branches 2205 2205
=======================================
Hits 9393 9393
Misses 17262 17262
Partials 489 489
☔ View full report in Codecov by Sentry. |
I confirm this build with |
I also confirm this build without the env var shows the debugging inspector 🤔 |
With latest changes I can confirm the changes are working as expected:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Blocked by #7345, will merge after resolution |
Blocked by #7347, will merge after resolution |
Kudos, SonarCloud Quality Gate passed! |
Description
This PR enables WebView debugging in android when either
RAMP_DEV_BUILD
orRAMP_INTERNAL_BUILD
environment variables are set.Manual testing steps
Create a production build in bitrise withRAMP_INTERNAL_BUILD
set totrue
Pre-merge author checklist
Pre-merge reviewer checklist