Refactor: Made kwargs keyword only for astroquery/utils/

Author: nkphysics

astroquery/utils/commons.py

@@ -280,7 +280,7 @@ def get_fits(self):
 
         return self._fits
 
-    def save_fits(self, savepath, link_cache='hard'):
+    def save_fits(self, savepath, *, link_cache='hard'):
         """
         Save a FITS file to savepath
 

astroquery/utils/docstr_chompers.py

@@ -10,7 +10,7 @@ def dec(fn):
     return dec
 
 
-def prepend_docstr_nosections(doc, sections=['Returns', ]):
+def prepend_docstr_nosections(doc, *, sections=['Returns', ]):
     """
     Decorator to prepend to the function's docstr after stripping out the
     list of sections provided (by default "Returns" only).

astroquery/utils/mocks.py

@@ -12,7 +12,7 @@ class MockResponse:
     A mocked/non-remote version of `astroquery.query.AstroResponse`
     """
 
-    def __init__(self, content=None, url=None, headers={}, content_type=None,
+    def __init__(self, content=None, *, url=None, headers={}, content_type=None,
                  stream=False, auth=None, status_code=200, verify=True,
                  allow_redirects=True, json=None):
         assert content is None or hasattr(content, 'decode')

astroquery/utils/process_asyncs.py

@@ -53,7 +53,7 @@ def newmethod(self, *args, **kwargs):
     return cls
 
 
-def async_to_sync_docstr(doc, returntype='table'):
+def async_to_sync_docstr(doc, *, returntype='table'):
     """
     Strip of the "Returns" component of a docstr and replace it with "Returns a
     table" code

astroquery/utils/progressbar.py

@@ -21,7 +21,7 @@ def chunk_report(bytes_so_far, chunk_size, total_size):
                          (bytes_so_far / 1024. ** 2))
 
 
-def chunk_read(response, chunk_size=1024, report_hook=None):
+def chunk_read(response, *, chunk_size=1024, report_hook=None):
     content_length = response.info().get('Content-Length')
     if content_length is None:
         total_size = 0
@@ -51,7 +51,7 @@ def chunk_read(response, chunk_size=1024, report_hook=None):
     return result_string
 
 
-def retrieve(url, outfile, opener=None, overwrite=False):
+def retrieve(url, outfile, *, opener=None, overwrite=False):
     """
     "retrieve" (i.e., download to file) a URL.
     """

astroquery/utils/schema.py

@@ -55,7 +55,7 @@ def validate(self, data):
 
 class Use:
 
-    def __init__(self, callable_, error=None):
+    def __init__(self, callable_, *, error=None):
         assert callable(callable_)
         self._callable = callable_
         self._error = error
@@ -97,7 +97,7 @@ def priority(s):
 
 class Schema:
 
-    def __init__(self, schema, error=None):
+    def __init__(self, schema, *, error=None):
         self._schema = schema
         self._error = error
 

astroquery/utils/timer.py

@@ -21,7 +21,7 @@
 __doctest_skip__ = ['timefunc']
 
 
-def timefunc(num_tries=1, verbose=True):
+def timefunc(*, num_tries=1, verbose=True):
     """Decorator to time a function or method.
 
     Parameters
@@ -207,7 +207,7 @@ def time_func(self, arglist):
             self._cache_time(arg)
 
     # FUTURE: Implement N^x * O(log(N)) fancy fitting.
-    def do_fit(self, model=None, fitter=None, power=1, min_datapoints=3):
+    def do_fit(self, *, model=None, fitter=None, power=1, min_datapoints=3):
         """Fit a function to the lists of arguments and
         their respective run time in the cache.
 
@@ -304,7 +304,7 @@ def predict_time(self, arg):
             self._cache_est[arg] = t_est
         return t_est
 
-    def plot(self, xscale='linear', yscale='linear', xlabeltext='args',
+    def plot(self, *, xscale='linear', yscale='linear', xlabeltext='args',
              save_as=''):  # pragma: no cover
         """Plot prediction.