make python 3.6 defualt runtime for click options instead of 3.5

[aaronayres35]

fixes #429

This PR simply bumps the runtime option default from 3.5 to 3.6 as 3.6 is what is listed in supported_combinations

[rahulporuri]

LGTM. Can you also update the docstring at the top of the module? The docstring still refers to 2.7 and 3.5 as the supported runtimes.

[aaronayres35]

LGTM. Can you also update the docstring at the top of the module? The docstring still refers to 2.7 and 3.5 as the supported runtimes.

I also change the docstring to say "currently supported toolkits are null, pyqt, and pyqt5" instead of "currently supported toolkits are null, pyqt, and wx" and wx is also not a part of supported_combinations atm.

[aaronayres35]

LGTM. Can you also update the docstring at the top of the module? The docstring still refers to 2.7 and 3.5 as the supported runtimes.

I also change the docstring to say "currently supported toolkits are null, pyqt, and pyqt5" instead of "currently supported toolkits are null, pyqt, and wx" and wx is also not a part of supported_combinations atm.

seems wx support was lost here: #384 . Should we open an issue to re-add wx support/ re-add it to CI?

[rahulporuri]

seems wx support was lost here: #384 . Should we open an issue to re-add wx support/ re-add it to CI?

we definitely should. I think there is a PR open that tries fixing the wx support.

[codecov-io]

Codecov Report

Merging #430 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #430   +/-   ##
=======================================
  Coverage   28.92%   28.92%           
=======================================
  Files         206      206           
  Lines       18246    18246           
  Branches     2466     2466           
=======================================
  Hits         5278     5278           
  Misses      12641    12641           
  Partials      327      327           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 375d52e...4816fed. Read the comment docs.

[aaronayres35]

seems wx support was lost here: #384 . Should we open an issue to re-add wx support/ re-add it to CI?

we definitely should. I think there is a PR open that tries fixing the wx support.

ah my mistake, it is still run on CI just as an allowed failure. We still would want to fix that/ add it back to the supported_combinations though