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.

• [x] astroquery/alfalfa #2661
• [x] astroquery/alma #2309
• [x] astroquery/astrometry_net #2661
• [x] astroquery/atomic
• [x] astroquery/besancon #2661
• [x] astroquery/cadc
• [x] astroquery/casda #2661
• [x] astroquery/cds #2661
• [ ] astroquery/cosmosim
• [x] astroquery/dace #2661
• [x] astroquery/esa/hsa
• [x] astroquery/esa/hubble #2597
• [x] astroquery/esa/iso
• [x] astroquery/esa/jwst
• [x] astroquery/esa/xmm_newton
• [x] astroquery/esasky
• [x] astroquery/eso #2661
• [x] astroquery/exoplanet_orbit_database #2661
• [x] astroquery/ferm i#2661
• [x] astroquery/gaia #2661
• [x] astroquery/gama #2661
• [x] astroquery/gemini #2661
• [x] astroquery/heasarc
• [x] astroquery/hips2fits #2661
• [x] astroquery/hitran #2661
• [x] astroquery/image_cutouts/first #2661
• [x] astroquery/imcce #2661
• [x] astroquery/ipac/irsa #2092
• [x] astroquery/ipac/irsa/ibe #2661
• [x] astroquery/ipac/irsa/irsa_dust #2661
• [x] astroquery/ipac/irsa/sha #2661
• [x] astroquery/ipac/ned #2661
• [x] astroquery/ipac/nexsci/nasa_exoplanet_archive
• [x] astroquery/jplhorizons #1802
• [x] astroquery/jplsbdb #2661
• [x] astroquery/jplspec #2661
• [x] astroquery/lamda #2661
• [x] astroquery/linelists/cdms #2661
• [x] astroquery/magpis #2661
• [ ] astroquery/mast #2317 This issue highlights that there are still todo for mast: https://github.com/astropy/astroquery/issues/2665
• [x] astroquery/mpc #2661
• [x] astroquery/nasa_ads #2661
• [x] astroquery/nist #2655
• [x] astroquery/nvas #2656
• [x] astroquery/oac #2661
• [x] astroquery/ogle #2661
• [x] astroquery/open_exoplanet_catalogue #2661
• [x] astroquery/sdss #2477 #2532
• [x] astroquery/simbad #2609
• [x] astroquery/skyview #2601
• [x] astroquery/splatalogue #2661
• [x] astroquery/svo_fps #2661
• [x] astroquery/template_module #2661
• [x] astroquery/ukidss #2661
• [x] astroquery/utils #2690
• [x] astroquery/utils/tap #2690
• [ ] astroquery/vamdc
• [x] astroquery/vizier #2610
• [x] astroquery/vo_conesearch
• [x] astroquery/vsa #2661
• [x] astroquery/wfau #2661
• [x] astroquery/xmatch #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!