-
-
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
docs: Update Appium documentation #11124
Conversation
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
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #11124 +/- ##
==========================================
+ Coverage 52.75% 52.91% +0.16%
==========================================
Files 1534 1545 +11
Lines 36777 36901 +124
Branches 4335 4352 +17
==========================================
+ Hits 19403 19528 +125
+ Misses 16058 16052 -6
- Partials 1316 1321 +5 ☔ View full report in Codecov by Sentry. |
This reverts commit 958abf4.
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
Quality Gate passedIssues Measures |
A bit late to comment as it's merged already, but I wanted to say it's nice work! |
Description
The goal of this task is to enhance the clarity of the Appium section, ensuring that everyone understands how tests are executed and triggered. The section needs refinement to provide clearer step-by-step instructions, so the process is easy to follow and all team members are aligned.
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist