From 7c503018234af0e5a1eb31ca78380833a750b65f Mon Sep 17 00:00:00 2001 From: Karim Nakad Date: Tue, 16 Jul 2019 08:47:42 -0700 Subject: [PATCH] change: enable chained-comparison Pylint check --- .pylintrc | 1 - src/sagemaker/parameter.py | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/.pylintrc b/.pylintrc index a8308f4400..4dd8f23304 100644 --- a/.pylintrc +++ b/.pylintrc @@ -92,7 +92,6 @@ disable= consider-using-in, # TODO: Consider merging comparisons with "in" too-many-public-methods, # TODO: Resolve consider-using-ternary, # TODO: Consider ternary expressions - chained-comparison, # TODO: Simplify chained comparison between operands too-many-branches, # TODO: Simplify or ignore as appropriate missing-docstring, # TODO: Fix missing docstring diff --git a/src/sagemaker/parameter.py b/src/sagemaker/parameter.py index 16372d80d4..152455c107 100644 --- a/src/sagemaker/parameter.py +++ b/src/sagemaker/parameter.py @@ -47,7 +47,7 @@ def is_valid(self, value): Returns: bool: True if valid, False otherwise. """ - return value >= self.min_value and value <= self.max_value + return self.min_value <= value <= self.max_value @classmethod def cast_to_type(cls, value):