search

egh / ledger-autosync

Synchronize your ledger-cli files with your bank.
GNU General Public License v3.0
273 stars 42 forks source link

Add argument to specify ofxclient config file #58

Closed 501st-alpha1 closed 6 years ago

501st-alpha1 commented 6 years ago

I have separate ofxclient.ini files for my business vs. personal accounts, and I got tired of renaming them back and forth, so I've adding a command-line argument allowing me to specify which one I want to use. (This will also let me store the files elsewhere and have one less thing in $HOME.)

Let me know if you'd like me to add a test for this, or if you have any other suggestions.

egh commented 6 years ago

This looks good. My only hesitation would be that we might want to set up our own config file at some point, so maybe using --ofxconfig and -o rather than --config and -c? Would that work?

501st-alpha1 commented 6 years ago

Yes that makes sense, and your suggestion sounds good. I've pushed another commit for that; let me know if you'd rather I squash them down into a single commit.

egh commented 6 years ago

Looks good to me! Thank you.