Add click 8.1.0 support

[zanieb]

Some deprecated code was removed in pallets/click#2130 resulting in the failed import of get_terminal_size from click. Per the recommendation in Click's release notes, we can import from shutil instead.

get_terminal_size can probably be dropped from Typer's interface in the future since it's been dropped from Click's, but I'll leave that decision to a future PR. This change is backwards compatible.

See https://click.palletsprojects.com/en/8.1.x/changes/#version-8-1-0
Closes #377

[jepler]

Thanks! This looks like a correct fix.

[genos]

I came to after experiencing the aforementioned bug just this afternoon, only to find y'all already had a fix in the works. Thanks!

[Ravi4moreyeahs]

Traceback (most recent call last):
File "/Users/moreyeahs/CheQ ONE/Dockerized/app/main.py", line 17, in
from bank.default import DefaultPatern
File "/Users/moreyeahs/CheQ ONE/Dockerized/app/bank/default.py", line 2, in
import spacy
File "/Users/moreyeahs/CheQ ONE/Dockerized/venv/lib/python3.10/site-packages/spacy/init.py", line 15, in
from .cli.info import info # noqa: F401
File "/Users/moreyeahs/CheQ ONE/Dockerized/venv/lib/python3.10/site-packages/spacy/cli/init.py", line 3, in
from ._util import app, setup_cli # noqa: F401
File "/Users/moreyeahs/CheQ ONE/Dockerized/venv/lib/python3.10/site-packages/spacy/cli/_util.py", line 9, in
import typer
File "/Users/moreyeahs/CheQ ONE/Dockerized/venv/lib/python3.10/site-packages/typer/init.py", line 12, in
from click.termui import get_terminal_size as get_terminal_size
ImportError: cannot import name 'get_terminal_size' from 'click.termui'

[danielbraun89]

This is a high priority bug fix! all new installations of typer are basically broken

[jepler]

If there's any way Adafruit can help get this merged and a release made, please @ me!

[austinbutler]

Confirmed working. Anyone using Poetry for their dependencies can use:

[tool.poetry.dependencies]
typer = { git = "https://github.com/madkinsz/typer.git", rev = "b6efa2f" }

[Andrew-Sheridan]

@tiangolo any chance you could take a look at this?

Thank you

[github-actions]

📝 Docs preview for commit b6efa2f at: https://6244897abaa0e80081318186--typertiangolo.netlify.app

[codecov]

Codecov Report

Merging #375 (b6efa2f) into master (6d8e954) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #375   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          252       252           
  Lines         5287      5287           
=========================================
  Hits          5287      5287           

Impacted Files	Coverage Δ
tests/__init__.py	100.00% <0.00%> (ø)

---

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 6d8e954...b6efa2f. Read the comment docs.

[zanieb]

Requires a black update to pass CI as it's affected by the same bug :)

Closing in favor of #380

[tiangolo]

Thanks for the work @madkinsz! And thanks everyone for the feedback. This was included in #380, and is now released in Typer 0.4.1. 🚀 🎉

[jepler]

Thank you @tiangolo