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

Inconsistency between documentation and code: * comments missing from csv rules #1953

Closed
sprig opened this issue Dec 2, 2022 · 2 comments
Closed
Labels
A-BUG Something wrong, confusing or sub-standard in the software, docs, or user experience. csv The csv file format, csv output format, or generally CSV-related. docs Documentation-related.

Comments

@sprig
Copy link

sprig commented Dec 2, 2022

Hello and thanks for creating and maintaining hledger!

I just spent some time trying to figure out what lines starting with * mean, in a rules file (that I wrote a while ago, moreover). Suspected and eventually confirmed to be comments by the below referenced line, nowhere in the docs is * mentioned as an allowable comment character, as far as I can tell, including in specification laid out in the same file. I did eventually find https://hledger.org/1.28/hledger.html#comments which allows for * comments in the journal, which I presume is why I tried the same in the rules file.

Definition:

commentcharp = oneOf (";#*" :: [Char])

Spec:
COMMENT-CHAR: # | ;

Documentation:
First paragraph of https://hledger.org/1.28/hledger.html#csv-rules

@sprig sprig added the A-BUG Something wrong, confusing or sub-standard in the software, docs, or user experience. label Dec 2, 2022
@simonmichael simonmichael added docs Documentation-related. csv The csv file format, csv output format, or generally CSV-related. labels Dec 6, 2022
@simonmichael
Copy link
Owner

Thanks! I have clarified the support for * comment lines in CSV rules, timeclock and timedot formats.

@simonmichael
Copy link
Owner

Though FWIW, supporting * comment lines seems no longer valuable. Nowadays, Emacs users like myself probably prefer using ledger-mode plus outshine-mode to fold journals, rather than switching to org mode and losing the benefits of ledger-mode. I still find it useful for timedot files, but that could change. If anyone wants to champion it, we could explore deprecating and eventually dropping this syntax.

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. csv The csv file format, csv output format, or generally CSV-related. docs Documentation-related.
Projects
None yet
Development

No branches or pull requests

2 participants