chore: remove no_extract arg from workbook download

Author: jorwoods

tableauserverclient/server/endpoint/workbooks_endpoint.py

@@ -182,9 +182,8 @@ def download(
         workbook_id: str,
         filepath: Optional[PathOrFileW] = None,
         include_extract: bool = True,
-        no_extract: Optional[bool] = None,
     ) -> str:
-        return self.download_revision(workbook_id, None, filepath, include_extract, no_extract)
+        return self.download_revision(workbook_id, None, filepath, include_extract, )
 
     # Get all views of workbook
     @api(version="2.0")
@@ -445,7 +444,6 @@ def download_revision(
         revision_number: Optional[str],
         filepath: Optional[PathOrFileW] = None,
         include_extract: bool = True,
-        no_extract: Optional[bool] = None,
     ) -> PathOrFileW:
         if not workbook_id:
             error = "Workbook ID undefined."
@@ -455,15 +453,6 @@ def download_revision(
         else:
             url = "{0}/{1}/revisions/{2}/content".format(self.baseurl, workbook_id, revision_number)
 
-        if no_extract is False or no_extract is True:
-            import warnings
-
-            warnings.warn(
-                "no_extract is deprecated, use include_extract instead.",
-                DeprecationWarning,
-            )
-            include_extract = not no_extract
-
         if not include_extract:
             url += "?includeExtract=False"