change: update linting - Githubissues

aws / sagemaker-inference-toolkit

Serve machine learning models within a 🐳 Docker container using 🧠 Amazon SageMaker.

Apache License 2.0

379 stars 82 forks source link

change: update linting #28

Closed ajaykarpur closed 4 years ago

ajaykarpur commented 4 years ago

Issue #, if available: Addresses comments from https://github.com/aws/sagemaker-inference-toolkit/pull/27

Description of changes:

Remove nonexistent directories from .pylintrc blacklist
Revert changes to DefaultInferenceHandler
Do not require docstring for private functions and classes
Add empty newlines around list in docstring

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.