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.

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

from_input method should load taskrc_location and data_location #58

Open
lyz-code opened this issue Aug 7, 2018 · 4 comments
Open

from_input method should load taskrc_location and data_location #58

lyz-code opened this issue Aug 7, 2018 · 4 comments

Comments

@lyz-code
Copy link
Contributor

lyz-code commented Aug 7, 2018

Hi, I'm writing some hooks and I noticed that the from_input method loads a TaskWarrior backend with the following configuration

backend = backends.TaskWarrior(data_location=hook_parent_dir)

Therefore it loads the data from the hook parent directory but the configuration from ~/.taskrc.

Now with the hooks V2 it would be as easy as to load those two values from sys.argv.

I'm willing to do a Pull Request if you are interested.

Please let me know

@xeruf
Copy link
Contributor

xeruf commented May 28, 2020

yes, please implement that :)

@lyz-code
Copy link
Contributor Author

lyz-code commented Jun 1, 2020

Sorry @xerus2000, due to the lack of activity of taskwarrior/tasklib I've migrated my efforts to pydo

@xeruf
Copy link
Contributor

xeruf commented Jun 2, 2020

Hmpf, that's sad. I have been working myself into taskwarrior now. I don't wanna switch again, but I'm already hitting pain points.

I do understand that the representation as text file isn't always optimal, but it is the UNIX philosophy and can be really useful if you work a lot with text-processing CLI tools - renaming tags or whatever can easily be done without even using taskwarrior.

@lyz-code
Copy link
Contributor Author

lyz-code commented Jun 3, 2020

I agree with you, but these reasons made me take the decision. Regarding the UNIX point, I think that as long as the application supports the import and export of it's data as json, it makes it compliant with the principle.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants