search

joecannatti / soji

Terminal Based Mindful Workday System
GNU Affero General Public License v3.0
75 stars 16 forks source link

Remove all crontab entries #20

Closed agbrook closed 7 years ago

agbrook commented 7 years ago

Before starting a new log entry, remove current and past crontab entries. This will allow for finishing tasks early as well as keep the crontab file cleaner. This should fix issue #18, I think.

joecannatti commented 7 years ago

Thanks! I will review the PR and merge soon.

On Nov 18, 2016 12:06 PM, "Andy Brook" notifications@github.com wrote:

Before starting a new log entry, remove current and past crontab entries. This will allow for finishing tasks early as well as keep the crontab file cleaner. This should fix issue #18

https://github.com/joecannatti/soji/issues/18, I think.

You can view, comment on, or merge this pull request online at:

https://github.com/joecannatti/soji/pull/20 Commit Summary

  • Remove all crontab entries

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/joecannatti/soji/pull/20, or mute the thread https://github.com/notifications/unsubscribe-auth/AAJvJn1VLGl2G-aL99TMrvMIqVXYm-Taks5q_ekPgaJpZM4K2w6e .

joecannatti commented 7 years ago

Hey @agbrook , I got confused apparently and thought this was already merged. Are you still using Soji? If you update this from master I can take a look now. I realize it's been a long time. Sorry.

agbrook commented 7 years ago

Hey @joecannatti No problem. I'm still using soji. I like having this patch in place. I did add an alias "alias sedit='vim $(soji note-path log)'", so that I can easily edit the log because I'm not great at always remembering to start a meeting or lunch when I actually start a meeting or take lunch.

I updated this merge request with the latest patches that have been merged in.