Replace unused variable in lambda expression#2160
Conversation
In Python it is common to indicate unused variables by choosing _ (underscore) as variable name. This commit replaces some instances of unused variables in lambda expressions to align with Python customs and also other test examples in the repository.
✅ Deploy Preview for selenium-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Code Suggestions ✨Explore these optional code suggestions:
|
As the example is explicitly on how side effects can be achieved the suggestion is imho a false positive. |
|
Congratulations on your first contribution to Selenium-docs @b2m 🎉 🎉 |
Great experience, feels like I should repeat it sometime 😉. Thank you @harsha509. |
User description
Description
In Python it is common to indicate unused variables by choosing _ (underscore) as variable name. This pull request replaces some instances of unused variables in lambda expressions to align with Python customs and also other test examples in the repository.
Types of changes
Checklist
PR Type
Enhancement
Description
Replaced unused variables in lambda expressions with
_for Python convention.Improved code readability and consistency with Python customs.
Changes walkthrough 📝
test_waits.py
Update lambda expressions to use `_` for unused variablesexamples/python/tests/waits/test_waits.py
dwith_in lambda expressions to indicate unused variables.test_explicitandtest_explicit_options) forbetter readability.