Added type hints/annotations to all methods

[y0urself]

Added type hints/annotations to all methods

Checklist:

• Tests
• CHANGELOG Entry
• Documentation N/A

[codecov]

Codecov Report

Merging #32 into master will increase coverage by 0.25%.
The diff coverage is 52.8%.

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
+ Coverage   40.31%   40.57%   +0.25%     
==========================================
  Files          12       12              
  Lines         439      456      +17     
==========================================
+ Hits          177      185       +8     
- Misses        262      271       +9

Impacted Files	Coverage Δ
autohooks/version.py	0% <0%> (ø)	⬆️
autohooks/cli/check.py	0% <0%> (ø)	⬆️
autohooks/precommit/run.py	0% <0%> (ø)	⬆️
autohooks/api/git.py	0% <0%> (ø)	⬆️
autohooks/cli/activate.py	0% <0%> (ø)	⬆️
autohooks/template.py	100% <100%> (ø)	⬆️
autohooks/install.py	66.66% <100%> (+1.8%)	⬆️
autohooks/api/path.py	75% <100%> (+5%)	⬆️
autohooks/terminal.py	71.42% <100%> (ø)	⬆️
autohooks/utils.py	82.35% <100%> (ø)	⬆️
... and 2 more

---

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 55a8384...35a4ee8. Read the comment docs.

[bjoernricks]

I don't know why you did remove Mode.get_effective_mode. But besides it looks very very good again. Thank you very much!

[y0urself]

I am relatively sure, that the function is not necessary in the one case it was used.
The if-statement does the same, as the method. The remaining cases will go into the else, with or without the function. Or have I missed something out?

[bjoernricks]

You are right currently the behavior is the same without the get_effective_mode method. But it has a semantic meaning which I don't want to remove (currently). Additionally it would have been better to remove the method in another PR.

[bjoernricks]

Thanks a lot!