add target_directory() docstring

pytube/cipher.py

@@ -144,7 +144,7 @@ def get_transform_map(js: str, var: str) -> Dict:
     return mapper
 
 
-def reverse(arr: List, b: Optional[Any]):
+def reverse(arr: List, _: Optional[Any]):
     """Reverse elements in a list.
 
     This function is equivalent to:

pytube/helpers.py

@@ -6,7 +6,7 @@ import os
 import pprint
 import re
 import warnings
-from typing import TypeVar, Callable
+from typing import TypeVar, Callable, Optional
 
 from pytube.exceptions import RegexMatchError
 
@@ -138,7 +138,17 @@ def deprecated(reason: str) -> Callable:
     return decorator
 
 
-def target_directory(output_path: str) -> str:
+def target_directory(output_path: Optional[str]) -> str:
+    """
+    Function for determining target directory of a download.
+    Returns an absolute path (if relative one given) or the current
+    path (if none given). Makes directory if it does not exist.
+
+    :type output_path: str
+        :rtype: str
+    :returns:
+        An absolute directory path as a string.
+    """
     if output_path:
         if not os.path.isabs(output_path):
             output_path = os.path.join(os.getcwd(), output_path)

pytube/query.py

@@ -296,7 +296,7 @@ class StreamQuery:
 
         """
         return (
-            self.filter(only_audio=True, codec=codec)
+            self.filter(only_audio=True, audio_codec=codec)
             .order_by("resolution")
             .asc()
             .last()

pytube/streams.py

@@ -72,9 +72,7 @@ class Stream:
         # Same as above, except for the format profile attributes.
         self.set_attributes_from_dict(self.fmt_profile)
 
-        # The player configuration which contains information like the video
-        # title.
-        # TODO(nficano): this should be moved to the monostate.
+        # The player configuration, contains info like the video title.
         self.player_config_args = player_config_args
 
         # 'video/webm; codecs="vp8, vorbis"' -> 'video/webm', ['vp8', 'vorbis']