docs version unclear (Mixing .csv and .ledger files) #385

Closed
rstacruz opened this Issue Aug 7, 2016 · 10 comments

Projects

None yet

2 participants

@rstacruz
rstacruz commented Aug 7, 2016 edited

Hey there, the docs say you can mix files of different formats (emphasis mine):

Multiple files
You can specify multiple -f/--file FILE options. This is like combining all the files into one, except they can have different formats. Also directives and aliases in one file do not affect subsequent files (if you need that, use the include directive instead).

But this doesn't seem to work:

# setup.ledger
2015/01/01 Hello
  A  P 1.00
  B
# cash.csv
2016/08/06, 957, Food
# cash.csv.rules
fields date, amount, description
amount P %2
account1 Expenses:Variable:Misc
account2 Assets:Cash
status *
@simonmichael
Owner

It works for me, but only using the development version of hledger, from git. I'm guessing you read http://hledger.org/manual.html but didn't notice that it's for this version. The "version: dev | 0.27" links at the top try to hint this. Does that explain it ? How could things be improved ?

$ hledger -f setup.ledger -f cash.csv print
using conversion rules file /Users/simon/src/hledger/385/cash.csv.rules
2015/01/01 Hello
    A        P 1.00
    B       P -1.00

2016/08/06 *  Food
    Assets:Cash                   P -957
    Expenses:Variable:Misc         P 957

~/src/hledger$ hledger-0.27.1 -f setup.ledger -f cash.csv print
using conversion rules file /Users/simon/src/hledger/385/cash.csv.rules
hledger-0.27.1: user error (CSV record ["2015/01/01 Hello"] has less than two fields)
@rstacruz
rstacruz commented Aug 7, 2016

The docs said it was for 0.27, and I was running 0.27.1 from homebrew :)

On Sun, Aug 7, 2016, 9:42 PM Simon Michael notifications@github.com wrote:

It works for me, but only using the development version of hledger, from
git. I'm guessing you read http://hledger.org/manual.html but didn't
notice that it's for this version. The "version: dev | 0.27" links at the
top try to hint this. Does that explain it ? How could things be improved ?

$ hledger -f setup.ledger -f cash.csv print
using conversion rules file /Users/simon/src/hledger/385/cash.csv.rules
2015/01/01 Hello
A P 1.00
B P -1.00

2016/08/06 * Food
Assets:Cash P -957
Expenses:Variable:Misc P 957
~/src/hledger$ hledger-0.27.1 -f setup.ledger -f cash.csv print
using conversion rules file /Users/simon/src/hledger/385/cash.csv.rules
hledger-0.27.1: user error (CSV record ["2015/01/01 Hello"] has less than two fields)


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#385 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEikc1kzDCHB4Atda0Muii39NZIl0n9ks5qdeDegaJpZM4JecRW
.

@simonmichael
Owner

Where do you see the docs saying they are for 0.27 ?

@rstacruz
rstacruz commented Aug 7, 2016

It says it's for version dev or 0.27 in the hledger website :)

On Sun, Aug 7, 2016, 10:06 PM Simon Michael notifications@github.com
wrote:

Where do you see the docs saying they are for 0.27 ?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#385 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEikbEK7VbrQ7cZlGBwtSJ7cYVlKY_Mks5qdeaCgaJpZM4JecRW
.

@simonmichael
Owner

Hmm. So, to be specific, you're looking at eg the top of http://hledger.org/manual.html ?

There, you see "version: dev | 0.27" which is my ultra-cheap docs version indicator/selector. "dev" is not a link, "0.27" is. In my mind, this means "you are looking at the docs for the unreleased development version. Click the link to see docs for that version instead".

Clicking 0.27 takes you to http://hledger.org/doc/0.27/manual.html, which is the doc you want. (This one does not have the version links at top because I didn't want to fiddle with the old docs.)

Making sense ? Suggestions/patches to make it clearer ?

@rstacruz
rstacruz commented Aug 7, 2016

Oh! Oops, yeah, that part was rather confusing indeed. I see now. I have no
suggestions on how to improve it though.

Thank you for hledger!

On Sun, Aug 7, 2016, 10:18 PM Simon Michael notifications@github.com
wrote:

Hmm. So, to be specific, you're looking at eg the top of
http://hledger.org/manual.html ?

There, you see "version: dev | 0.27" which is my ultra-cheap docs version
indicator/selector. "dev" is not a link, "0.27" is. In my mind, this means
"you are looking at the docs for the unreleased development version. Click
the link to see docs for that version instead".

Clicking 0.27 takes you to http://hledger.org/doc/0.27/manual.html, which
is the doc you want. (This one does not have the version links at top
because I didn't want to fiddle with the old docs.)

Making sense ? Suggestions/patches to make it clearer ?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#385 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEikdgUpBt-IfTYyAr9fLFGxZBKzoklks5qdelKgaJpZM4JecRW
.

@simonmichael
Owner

Working on a better doc version selector. Thanks for the reports!

@rstacruz
rstacruz commented Aug 7, 2016

My impression was it was an indicator like "Available on versions: dev,
0.27, 0.26 and 0.25".

On Sun, Aug 7, 2016, 11:19 PM Simon Michael notifications@github.com
wrote:

Working on a better doc version selector. Thanks for the reports!


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#385 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEikbIPBGEdTEZ1sgHDZPTBpBWJHFq4ks5qdfd5gaJpZM4JecRW
.

@simonmichael simonmichael changed the title from Mixing .csv and .ledger files to docs version unclear (Mixing .csv and .ledger files) Aug 7, 2016
@simonmichael
Owner

Closed this accidentally. But hopefully it's now clear enough which version of web manual you are looking at and how to get to the right one. If not let me know.

@simonmichael simonmichael added the A BUG label Aug 7, 2016
@simonmichael
Owner

PS you'll have to reload the hledger.org pages once in your browser to see the latest.

@mstksg mstksg added a commit to mstksg/hledger that referenced this issue Feb 3, 2017
@simonmichael @mstksg + mstksg doc: 0.27 manual version indicator fixes #385 55082ea
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment