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

1.32: import --catchup no longer writes .latest files #2156

Closed
ishmaelavila opened this issue Jan 18, 2024 · 2 comments
Closed

1.32: import --catchup no longer writes .latest files #2156

ishmaelavila opened this issue Jan 18, 2024 · 2 comments
Labels
A-BUG Something wrong, confusing or sub-standard in the software, docs, or user experience. bounty Thar's some kind o' loot on offer.. csv The csv file format, csv output format, or generally CSV-related. impact3 Affects just a few users. regression A backwards step, indicating a weakness in our QA. We don't like these. severity4 Major usability/doc bug, crash, or any regression.

Comments

@ishmaelavila
Copy link

ishmaelavila commented Jan 18, 2024

When I attempt to import a CSV from my bank and I specify the --catchup option, the command runs successfully, however, the .latest.sample.csv state file is not written. The existence of the file and it's content do not seem to have an impact on whether the command updates/creates the file.

I've been able to reproduce this issue reliably on versions above 1.31. The issue seems to have been introduced in 1.32 and I think it may be a regression as I couldn't find anything in the release notes suggesting it was an intentional change.

Please let me know if you need anything else to reproduce or if this is a problem on my end. Thanks!

$ hledger --version
hledger 1.32.2, linux-x86_64

Steps To Reproduce

Note, the commands assume you have the LEDGER_FILE environmental variable specified.

  1. Install hledger >=1.32
  2. Create a CSV that contains valid transactions for importing
  3. Run hledger import --catchup sample.csv
  4. Observe that the command runs succesfully
  5. Look for .latest.sample.csv to see if it was created or updated
  6. Run the import again and observe it continues to say the CSV was marked as caught up rather than saying no new transactions found in CSV

Expected Result

The command finishes successfully, prints the number of transactions skipped, and updated the .latest.sample.csv file with the latest dates.

Notes

When I run the import command without --catchup, the journal file is updated and the .latest.sample.csv file is created/updated correctly. The --catchup option works for me in version1.31 so I'll continue using that for now.

@ishmaelavila ishmaelavila added the A-BUG Something wrong, confusing or sub-standard in the software, docs, or user experience. label Jan 18, 2024
@simonmichael simonmichael added csv The csv file format, csv output format, or generally CSV-related. regression A backwards step, indicating a weakness in our QA. We don't like these. labels Jan 18, 2024
@simonmichael simonmichael changed the title .latest.*.csv files are not being written when importing CSVs using the --catchup option 1.32: import --catchup no longer writes .latest files Jan 18, 2024
@simonmichael
Copy link
Owner

simonmichael commented Jan 18, 2024

Confirmed, thanks for the report!

Claim your https://hledger.org/REGRESSIONS.html#regression-bounty when convenient. Per new policy, it's $50 to the finder and (if someone other than me fixes it) $50 to the fixer.

@simonmichael simonmichael added severity2 Minor to moderate usability/doc bug, reasonably easy to avoid or tolerate. impact3 Affects just a few users. severity4 Major usability/doc bug, crash, or any regression. and removed severity2 Minor to moderate usability/doc bug, reasonably easy to avoid or tolerate. labels Jan 18, 2024
@simonmichael simonmichael pinned this issue Jan 18, 2024
@simonmichael simonmichael added bounty Thar's some kind o' loot on offer.. needs-release This should probably be shipped in a release soon. labels Jan 19, 2024
@simonmichael
Copy link
Owner

Fixed in master.

@simonmichael simonmichael removed the needs-release This should probably be shipped in a release soon. label Jan 28, 2024
@simonmichael simonmichael unpinned this issue Jan 28, 2024
adept pushed a commit to adept/hledger that referenced this issue Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-BUG Something wrong, confusing or sub-standard in the software, docs, or user experience. bounty Thar's some kind o' loot on offer.. csv The csv file format, csv output format, or generally CSV-related. impact3 Affects just a few users. regression A backwards step, indicating a weakness in our QA. We don't like these. severity4 Major usability/doc bug, crash, or any regression.
Projects
None yet
Development

No branches or pull requests

2 participants