MAINT: making optional kwargs keyword only for skyview

Author: bsipocz

astroquery/skyview/core.py

@@ -85,7 +85,7 @@ def _submit_form(self, input=None, cache=True):
         response.raise_for_status()
         return response
 
-    def get_images(self, position, survey, coordinates=None, projection=None,
+    def get_images(self, position, survey, *, coordinates=None, projection=None,
                    pixels=None, scaling=None, sampler=None, resolver=None,
                    deedger=None, lut=None, grid=None, gridlabels=None,
                    radius=None, height=None, width=None, cache=True,
@@ -198,18 +198,16 @@ def get_images(self, position, survey, coordinates=None, projection=None,
         A list of `~astropy.io.fits.HDUList` objects.
 
         """
-        readable_objects = self.get_images_async(position, survey, coordinates,
-                                                 projection, pixels, scaling,
-                                                 sampler, resolver, deedger,
-                                                 lut, grid, gridlabels,
-                                                 radius=radius, height=height,
-                                                 width=width,
-                                                 cache=cache,
-                                                 show_progress=show_progress)
+        readable_objects = self.get_images_async(position, survey, coordinates=coordinates,
+                                                 projection=projection, pixels=pixels, scaling=scaling,
+                                                 sampler=sampler, resolver=resolver, deedger=deedger,
+                                                 lut=lut, grid=grid, gridlabels=gridlabels,
+                                                 radius=radius, height=height, width=width,
+                                                 cache=cache, show_progress=show_progress)
         return [obj.get_fits() for obj in readable_objects]
 
     @prepend_docstr_nosections(get_images.__doc__)
-    def get_images_async(self, position, survey, coordinates=None,
+    def get_images_async(self, position, survey, *, coordinates=None,
                          projection=None, pixels=None, scaling=None,
                          sampler=None, resolver=None, deedger=None, lut=None,
                          grid=None, gridlabels=None, radius=None, height=None,
@@ -219,18 +217,18 @@ def get_images_async(self, position, survey, coordinates=None,
         -------
         A list of context-managers that yield readable file-like objects
         """
-        image_urls = self.get_image_list(position, survey, coordinates,
-                                         projection, pixels, scaling, sampler,
-                                         resolver, deedger, lut, grid,
-                                         gridlabels, radius=radius,
+        image_urls = self.get_image_list(position, survey, coordinates=coordinates,
+                                         projection=projection, pixels=pixels, scaling=scaling, sampler=sampler,
+                                         resolver=resolver, deedger=deedger, lut=lut, grid=grid,
+                                         gridlabels=gridlabels, radius=radius,
                                          height=height, width=width,
                                          cache=cache)
         return [commons.FileContainer(url, encoding='binary',
                                       show_progress=show_progress)
                 for url in image_urls]
 
     @prepend_docstr_nosections(get_images.__doc__, sections=['Returns', 'Examples'])
-    def get_image_list(self, position, survey, coordinates=None,
+    def get_image_list(self, position, survey, *, coordinates=None,
                        projection=None, pixels=None, scaling=None,
                        sampler=None, resolver=None, deedger=None, lut=None,
                        grid=None, gridlabels=None, radius=None, width=None,