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

Rearrange get_files function to return better CLI output #58

Merged
merged 1 commit into from
Oct 26, 2017

Conversation

nocalla
Copy link
Member

@nocalla nocalla commented Oct 25, 2017

Remove output about missing directory when no files found in location matching section criteria. Makes for cleaner output. Removed source location from BOI so it will follow default behaviour.

Issue #48

Remove output about missing directory when no files found in location matching section criteria. Makes for cleaner output. Removed source location from BOI so it will follow default behaviour.
@torbengb
Copy link
Member

Tested this, see my comment over here.

@nocalla
Copy link
Member Author

nocalla commented Oct 25, 2017

Changed things around a bit. Example output below.

python bank2ynab.py
Format: AT Raiffeisen Bank
Can't find: c:\users\example-username\Downloads
Trying: D:\Users\nocalla\Downloads
Parsing file: elbi_umsaetze_ (2).csv
Writing file: fixed_elbi_umsaetze_ (2).csv
Removing file: elbi_umsaetze_ (2).csv
Done!
Format: IE Bank of Ireland
Can't find: c:\users\example-username\Downloads
Trying: D:\Users\nocalla\Downloads
Parsing file: test_BOI_TransactionExport.csv
Writing file: fixed_test_BOI_TransactionExport.csv
Removing file: test_BOI_TransactionExport.csv
Done!
2 files processed.

@nocalla nocalla merged commit bd81a2f into master Oct 26, 2017
@nocalla nocalla deleted the better-output branch October 26, 2017 18:43
@nocalla nocalla restored the better-output branch October 26, 2017 19:02
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

Successfully merging this pull request may close these issues.

None yet

2 participants