search

janik6n / jetzt

Light-weight project manager for Python projects.
MIT License
9 stars 0 forks source link

Bug in jetzt --remove #14

Closed janik6n closed 5 years ago

janik6n commented 5 years ago

Dash is not handled correctly:

Example:

> jetzt --remove
Which of these would you like to remove?  [PROD] python-dateutil
Removing PROD dependency python...
Skipping python as it is not installed.
Could not remove dependency python from jetzt_metadata.json (dependencies/python), please remove manually.
Removal of package python is ready.
janik6n commented 5 years ago

In cli.py, line 222:

Fix (also note the group name): match_object = re.match(r'^\[(?P<dep_type>\w+)\]\s+(?P<pkg_to_remove>[\w\-]+).*$', pkg_name)

Same issue on line 171.

janik6n commented 5 years ago

Fixed in https://github.com/janikarh/jetzt/commit/fc474c0708f41aae4eff41b00ee6896ea69eefec.