Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove --add and --rm CLI args. #156

Closed
alichtman opened this issue Oct 28, 2018 · 2 comments
Closed

Remove --add and --rm CLI args. #156

alichtman opened this issue Oct 28, 2018 · 2 comments
Assignees
@alichtman
Labels
feature New feature or request good first issue Good for newcomers
Milestone
v2.0

Comments

@alichtman
Copy link
Owner

--add config PATH DEST should be valid.

Work needs to be done in config.py.
@alichtman alichtman added feature New feature or request good first issue Good for newcomers labels Oct 28, 2018
@alichtman alichtman mentioned this issue Oct 28, 2018
Merged
@alichtman
Copy link
Owner Author

Minor bug fixes (and one potentially major issue) and then this can be merged.

@alichtman alichtman self-assigned this Oct 28, 2018
@alichtman alichtman added this to the v2.0 milestone Oct 28, 2018
@alichtman
Copy link
Owner Author

New gameplan: Remove --add and --rm args. We're making this an interface exclusive feature.

@alichtman alichtman changed the title Add 3rd input for --add Rm --add and --rm CLI args. Oct 29, 2018
@alichtman alichtman changed the title Rm --add and --rm CLI args. Remove --add and --rm CLI args. Oct 29, 2018
alichtman added a commit that referenced this issue Oct 30, 2018
@alichtman
Add --add and --rm path to Action Menu (#157)
62d36de 
#156 #140 

Related to #129
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alichtman alichtman

Labels
feature New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
v2.0
Development

No branches or pull requests
1 participant
@alichtman