opensearch-project / opensearch-py

Python Client for OpenSearch
https://opensearch.org/docs/latest/clients/python/
Apache License 2.0
359 stars 178 forks source link

Pylint #620

Closed sejal-verma closed 11 months ago

sejal-verma commented 11 months ago

Description

Enables lints in setup.cfg

Issues Resolved

Enables remaining pylint defaults

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

saimedhi commented 11 months ago

@sejal-verma, DCO check is failing and also changelog entry is missing.

codecov[bot] commented 11 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (db61b59) 72.09% compared to head (9f5d289) 72.12%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #620 +/- ## ========================================== + Coverage 72.09% 72.12% +0.02% ========================================== Files 89 89 Lines 7939 7939 ========================================== + Hits 5724 5726 +2 + Misses 2215 2213 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

dblock commented 11 months ago

Closing this, please reopen addressing https://github.com/opensearch-project/opensearch-py/pull/620#pullrequestreview-1756250739.