Codecov Report

Patch coverage: 100.00% and project coverage change: +0.05 🎉

Comparison is base (48e9ce5) 98.02% compared to head (a60a9de) 98.08%.

@@            Coverage Diff             @@
##           master     #680      +/-   ##
==========================================
+ Coverage   98.02%   98.08%   +0.05%     
==========================================
  Files          40       40              
  Lines        1719     1720       +1     
==========================================
+ Hits         1685     1687       +2     
+ Misses         34       33       -1     

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Thanks for this @jakob-keller, @woile. 🙇 I guess this will be in the next release within a few weeks judging from the release cadence here.

In general, argcomplete seems to follow semver, would it make sense to relax this to v2 entirely? Not sure if it was intentional in 92b1e89 to restrict minor versions.

This sometimes gets installed in environments along with other dependencies so just to reduce the number of times we get conflicts.

In general, argcomplete seems to follow semver, would it make sense to relax this to v2 entirely? Not sure if it was intentional in 92b1e89 to restrict minor versions.

This sometimes gets installed in environments along with other dependencies so just to reduce the number of times we get conflicts.

I agree. My intention here was to relax the restriction without questioning the generally restrictive way dependencies are specified in this repo. But this would be a great request, i.e. a separate issue IMO.

Thanks @jakob-keller. I've opened #683, feel free to chime in :)