[DevOps]: removing the other python version so builds succeed, and cleaning up the use of locals

Author: jpetrucciani

.travis.yml

@@ -1,7 +1,6 @@
 language: python
 dist: xenial
 python:
-  - '3.5'
   - '3.6'
 install:
   - pip install -r requirements.txt

pyclickup/globals.py

@@ -3,7 +3,7 @@
 """
 
 
-__version__ = "0.0.6"
+__version__ = "0.0.7"
 
 
 LIBRARY = "pyclickup"

pyclickup/models/client.py

@@ -6,7 +6,7 @@
 from pyclickup.globals import __version__, API_URL, LIBRARY, TEST_TOKEN, TEST_API_URL
 from pyclickup.models import User, Task, Team
 from pyclickup.models.error import RateLimited
-from pyclickup.utils.text import datetime_to_ts
+from pyclickup.utils.text import datetime_to_ts, filter_locals
 
 
 def test_client():
@@ -117,10 +117,10 @@ def _get_tasks(
         **kwargs
     ):
         """fetches the tasks according to the given options"""
-        params = locals()
+        params = filter_locals(locals(), extras=["team_id"])
 
         for option in self.task_boolean_options:
-            if params[option] is not None:
+            if option in params:
                 params[option] = str(params[option]).lower()
 
         options = [
@@ -130,7 +130,6 @@ def _get_tasks(
                 ",".join(params[x]) if x in self.task_list_options else params[x],
             )
             for x in params
-            if params[x] is not None and x not in ["team_id", "self", "kwargs"]
         ]
         path = "team/{}/task?{}".format(team_id, "&".join(options))
         return [Task(x, client=self) for x in self.get(path)["tasks"]]

pyclickup/utils/text.py

@@ -7,6 +7,7 @@
 
 FIRST_CAP = re.compile("(.)([A-Z][a-z]+)")
 ALL_CAP = re.compile("([a-z0-9])([A-Z])")
+LOCALS_FILTER = ["self", "kwargs"]
 
 
 def snakeify(text):
@@ -23,3 +24,15 @@ def ts_to_datetime(timestamp):
 def datetime_to_ts(date_object):
     """converts a datetime to a posix x1000 timestamp"""
     return int(date_object.timestamp() * 1000)
+
+
+def filter_locals(local_variables, extras=None):
+    """filters out builtin variables in the local scope and returns locals as a dict"""
+    var_filter = LOCALS_FILTER.copy()
+    if extras and isinstance(extras, list):
+        var_filter += extras
+    return {
+        x: local_variables[x]
+        for x in local_variables
+        if local_variables[x] is not None and x not in var_filter
+    }

setup.py

@@ -24,7 +24,6 @@
     packages=find_packages(),
     classifiers=[
         "Programming Language :: Python :: 3",
-        "Programming Language :: Python :: 3.4",
         "Programming Language :: Python :: 3.5",
         "Programming Language :: Python :: 3.6",
         "Programming Language :: Python :: 3.7",