Merge branch 'release/0.1.0'.

.vscode/settings.json

@@ -0,0 +1,5 @@
+{
+    "python.linting.enabled": true,
+    "python.linting.pylintUseMinimalCheckers": false,
+    "python.linting.pylintEnabled": true
+}

Makefile

@@ -7,6 +7,9 @@ init-dev: init
 test:
 	python3 -m unittest
 
+lint:
+	python3 -m pylint src/autodesk_forge_sdk
+
 build:
 	python3 -m build
 

README.md

@@ -17,7 +17,7 @@ import os
 from autodesk_forge_sdk import AuthenticationClient, Scope
 
 client = AuthenticationClient()
-auth = client.authenticate(os.environ["FORGE_CLIENT_ID"], os.environ["FORGE_CLIENT_SECRET"], [Scope.ViewablesRead])
+auth = client.authenticate(os.environ["FORGE_CLIENT_ID"], os.environ["FORGE_CLIENT_SECRET"], [Scope.VIEWABLES_READ])
 print(auth["access_token"])
 ```
 

requirements-dev.txt

@@ -1,3 +1,4 @@
+pylint
 build
 twine
 wheel

setup.py

@@ -5,7 +5,7 @@
 
 setuptools.setup(
     name="autodesk-forge-sdk",
-    version="0.0.4",
+    version="0.1.0",
     author="Petr Broz",
     author_email="[email protected]",
     description="Unofficial Autodesk Forge SDK for Python.",

src/autodesk_forge_sdk/__init__.py

@@ -1,3 +1,8 @@
-from .auth import AuthenticationClient, Scope, TokenProviderInterface, SimpleTokenProvider, OAuthTokenProvider
+"""
+Clients for communicating with different Autodesk Forge services.
+"""
+
+from .auth import AuthenticationClient, Scope, get_authorization_url
+from .auth import TokenProviderInterface, SimpleTokenProvider, OAuthTokenProvider
 from .oss import OSSClient
-from .md import ModelDerivativeClient, urnify
+from .md import ModelDerivativeClient, urnify

src/autodesk_forge_sdk/base.py

@@ -1,50 +1,62 @@
+"""
+Helper classes used by other API clients.
+"""
+
 import requests
 
-class BaseClient(object):
+
+class BaseClient:
+    """
+    Base client for accessing web-based APIs.
+    """
     def __init__(self, base_url: str):
         self.base_url = base_url
 
     def _resolve_url(self, url: str) -> str:
-        if url.startswith('/'):
+        if url.startswith("/"):
             url = self.base_url + url
         return url
 
-    def _get(self, url: str, params: dict=None, headers: dict=None) -> requests.Response:
+    def _get(self, url: str, **kwargs) -> requests.Response:
         url = self._resolve_url(url)
-        response = requests.get(url, params=params, headers=headers)
+        response = requests.get(url, **kwargs)
         response.raise_for_status()
         return response
 
-    def _post(self, url: str, form: dict=None, json: dict=None, buff=None, params: dict=None, headers: dict=None) -> requests.Response:
+    def _post(
+        self, url: str, form: dict = None, json: dict = None, buff=None, **kwargs
+    ) -> requests.Response:
         url = self._resolve_url(url)
         response = None
         if form:
-            response = requests.post(url, data=form, params=params, headers=headers)
+            response = requests.post(url, data=form, **kwargs)
         elif form:
-            response = requests.post(url, data=buff, params=params, headers=headers)
+            response = requests.post(url, data=buff, **kwargs)
         elif json:
-            response = requests.post(url, json=json, params=params, headers=headers)
+            response = requests.post(url, json=json, **kwargs)
         else:
-            response = requests.post(url, params=params, headers=headers)
+            response = requests.post(url, **kwargs)
         response.raise_for_status()
         return response
 
-    def _put(self, url: str, form: dict=None, json: dict=None, buff=None, params: dict=None, headers: dict=None) -> requests.Response:
+    def _put(
+        self, url: str, form: dict = None, json: dict = None, buff=None, **kwargs
+    ) -> requests.Response:
         url = self._resolve_url(url)
         response = None
         if form:
-            response = requests.put(url, data=form, params=params, headers=headers)
+            response = requests.put(url, data=form, **kwargs)
         elif buff:
-            response = requests.put(url, data=buff, params=params, headers=headers)
+            response = requests.put(url, data=buff, **kwargs)
         elif json:
-            response = requests.put(url, json=json, params=params, headers=headers)
+            response = requests.put(url, json=json, **kwargs)
         else:
-            response = requests.put(url, params=params, headers=headers)
+            response = requests.put(url, **kwargs)
         response.raise_for_status()
         return response
- 
-    def _delete(self, url: str, params: dict=None, headers: dict=None) -> requests.Response:
+
+    def _delete(self, url: str, **kwargs) -> requests.Response:
         url = self._resolve_url(url)
-        response = requests.delete(url, params=params, headers=headers)
+        response = requests.delete(url, **kwargs)
         response.raise_for_status()
-        return response
+        return response