fix: Fix Code style issues to match Java standards

ashwithpoojary98 commented 11 months ago

Change list

Added HTML tags to doc links
Added a private constructor to restrict the creation of new objects of the class
Removed extra parentheses from lambda expressions
Renamed class and method variables to be more descriptive
Added override annotations to methods that override the parent class

Types of changes

What types of changes are you proposing/introducing to Java client? Put an x in the boxes that apply

[x] No changes in production code.
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Details

To meet Java code standards, I made a few changes to method and class-level variables: Added a Lombok no-arg constructor with private access to restrict the user from creating a new object of the class, since this class consists of all static methods. Added override annotations to improve the readability of methods and removed additional code from lambda expressions.

linux-foundation-easycla[bot] commented 11 months ago

The committers listed above are authorized under a signed CLA.

:white_check_mark: login: ashwithpoojary98 (5afab9089dc6f20939b1f33e7e4b43a4daf3ce91, 454a74c0fe8d0ce58c7ec4d26c53b8a9f0edb962, d71cc8359d03aa83182f3595b1474ac31d98bc53, e6e0bb61244c7001122e20d72e3529e86a3a213f, b9e182deef1f7a51f9f67dd9942b0a1a99eda900, b6aee49aac8a4a1cb9044e4858534571edbe03c2, a24f657e5c322e45850d5f1a6ec4d7db95b3579c, 8a9f755c26955c05da3c1c954aadd67b47e64377, f83fa5ee0c5aa1a1e75fe2b1500e19ce39f6e4dd, e578c0ddbdff2145ea4fc2f685ef4c05ccfa9911, 50c35bf9f0e53480b4bf99688b0631208df3d5b5, c528f0ce702edff5fa698d5543dd69fd27681b36, 32fc2e9d466baf38e31644a78e563fe529c07dff, ab425541fbb021a86736ba137933e2bbed8ad336, fb19d2dacee14e180790e215e9f660a1140adb1d)

mykola-mokhnach commented 11 months ago

Thank you @ashwithpoojary98 Please rename the PR to follow https://www.conventionalcommits.org/en/v1.0.0/ Also please make sure you have signed the CLA

ashwithpoojary98 commented 11 months ago

@mykola-mokhnach Thanks for reviewing the PR. I updated the code as per your comments. Please review.

ashwithpoojary98 commented 11 months ago

updated the workflow issue

ashwithpoojary98 commented 11 months ago

Updated workflow issue

ashwithpoojary98 commented 11 months ago

The Boolean check type casting issue was reverted.

ashwithpoojary98 commented 11 months ago

All checks have passed!!

appium / java-client