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

Add an experimental rundir option #76

Merged
merged 4 commits into from
Feb 6, 2020
Merged

Add an experimental rundir option #76

merged 4 commits into from
Feb 6, 2020

Conversation

apauley
Copy link
Owner

@apauley apauley commented Feb 6, 2020

The experimental rundir is an attempt to restrict hledger-flow into processing just a subset of files, primarily to quickly get feedback/failures while adding new accounts to an existing set of accounts.

The use case has been described in issue #64.

It is experimental, because the only problem it currently solves is getting hledger-flow to fail fast, and one of the current side effects of doing so is that the generated include files are then written to only include the subset of files that were processed. But as soon as you do a full run again, the include files will be correctly re-generated as before.

Some thought needs to go into properly solving the need for fast failure without causing surprising results. But for now this change should make one of the workarounds currently in use obsolete, namely the use of an old v0.11.1.2 binary which accidentally processed only a subset of files when run in a deeper subdirectory.

If we can arrive at some behaviour that "does the right thing" while only processing a subset of files, then we can consider enabling this by default whenever the basedir is pointed to a deeper subdirectory.

hledger-flow import --help
Usage: hledger-flow import [BASEDIR] [--experimental-rundir ARG]
  Converts electronic transactions into categorised journal files

Available options:
  BASEDIR                  The hledger-flow base directory
  --experimental-rundir ARG
                           A subdirectory of the base where the command will
                           restrict itself to
  -h,--help                Show this help text

@apauley apauley merged commit ced0b70 into master Feb 6, 2020
@apauley apauley deleted the rundir branch February 6, 2020 21:04
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.

1 participant