Use keyword only arguments

[bsipocz]

Many functionalities in astroquery have numerous arguments. Many of those are optional kwarg atm, and I propose to switch to to be kwarg only. I think that would help

We've seen several cases in the past where adding new kwargs ended in awkwardness as they needed to be added to the last position to keep the API backwards compatible for position arg usage.

• astroquery/alfalfa Refactor: Make kwargs keyword only  #2661
• astroquery/alma Alma deprecation cleanup #2309
• astroquery/astrometry_net Refactor: Make kwargs keyword only  #2661
• astroquery/atomic
• astroquery/besancon Refactor: Make kwargs keyword only  #2661
• astroquery/cadc
• astroquery/casda Refactor: Make kwargs keyword only  #2661
• astroquery/cds Refactor: Make kwargs keyword only  #2661
• astroquery/cosmosim
• astroquery/dace Refactor: Make kwargs keyword only  #2661
• astroquery/esa/hsa
• astroquery/esa/hubble eHST queries and data using TAP and minor fixes in ESA modules #2597
• astroquery/esa/iso
• astroquery/esa/jwst
• astroquery/esa/xmm_newton
• astroquery/esasky
• astroquery/eso Refactor: Make kwargs keyword only  #2661
• astroquery/exoplanet_orbit_database Refactor: Make kwargs keyword only  #2661
• astroquery/ferm i#2661
• astroquery/gaia Refactor: Make kwargs keyword only  #2661
• astroquery/gama Refactor: Make kwargs keyword only  #2661
• astroquery/gemini Refactor: Make kwargs keyword only  #2661
• astroquery/heasarc
• astroquery/hips2fits Refactor: Make kwargs keyword only  #2661
• astroquery/hitran Refactor: Make kwargs keyword only  #2661
• astroquery/image_cutouts/first Refactor: Make kwargs keyword only  #2661
• astroquery/imcce Refactor: Make kwargs keyword only  #2661
• astroquery/ipac/irsa Cleanup Irsa module #2092
• astroquery/ipac/irsa/ibe Refactor: Make kwargs keyword only  #2661
• astroquery/ipac/irsa/irsa_dust Refactor: Make kwargs keyword only  #2661
• astroquery/ipac/irsa/sha Refactor: Make kwargs keyword only  #2661
• astroquery/ipac/ned Refactor: Make kwargs keyword only  #2661
• astroquery/ipac/nexsci/nasa_exoplanet_archive
• astroquery/jplhorizons Add optional_settings param to JPL Horizon Ephemerides #1802
• astroquery/jplsbdb Refactor: Make kwargs keyword only  #2661
• astroquery/jplspec Refactor: Make kwargs keyword only  #2661
• astroquery/lamda Refactor: Make kwargs keyword only  #2661
• astroquery/linelists/cdms Refactor: Make kwargs keyword only  #2661
• astroquery/magpis Refactor: Make kwargs keyword only  #2661
• astroquery/mast Make astroquery.mast methods take explicit kwargs  #2317 This issue highlights that there are still todo for mast: DOC: make signatures more explicit in MAST docstrings #2665
• astroquery/mpc Refactor: Make kwargs keyword only  #2661
• astroquery/nasa_ads Refactor: Make kwargs keyword only  #2661
• astroquery/nist Refactor: Make NIST keyword only #2655
• astroquery/nvas Refactor: Made NVAS optional kwargs keyword only #2656
• astroquery/oac Refactor: Make kwargs keyword only  #2661
• astroquery/ogle Refactor: Make kwargs keyword only  #2661
• astroquery/open_exoplanet_catalogue Refactor: Make kwargs keyword only  #2661
• astroquery/sdss Convert query_region to cone search #2477 Restore get_query_payload to all methods #2532
• astroquery/simbad Refactor: SIMBAD keyword-only #2609
• astroquery/skyview BUG: fix skyview radius #2601
• astroquery/splatalogue Refactor: Make kwargs keyword only  #2661
• astroquery/svo_fps Refactor: Make kwargs keyword only  #2661
• astroquery/template_module Refactor: Make kwargs keyword only  #2661
• astroquery/ukidss Refactor: Make kwargs keyword only  #2661
• astroquery/utils Make utils and utis/tap kwargs keyword only #2690
• astroquery/utils/tap Make utils and utis/tap kwargs keyword only #2690
• astroquery/vamdc
• astroquery/vizier Refactor: make Vizier kwargs keyword-only #2610
• astroquery/vo_conesearch
• astroquery/vsa Refactor: Make kwargs keyword only  #2661
• astroquery/wfau Refactor: Make kwargs keyword only  #2661
• astroquery/xmatch Refactor: Make kwargs keyword only  #2661

[bsipocz]

OK, so a lot of time has passed, and we moved a significant fraction of the modules to use keyword-only arguments. I would propose to try to make this change for all the modules for the next release.

[bsipocz]

@ceb8 @keflavich - It would be nice to go through this list for the next tagged release. Technically the changes are trivial, we only need to be careful to also change the tests and docs.

[bsipocz]

This is now almost fully done, the only modules remaining are having their own issues, and tracked separately, or are already deprecated for removal.

Enormous thanks to @nkphysics!