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.

Sign up for GitHub

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

Jump to bottom

slight regression in debug output #1395

Open
simonmichael opened this issue Nov 17, 2020 · 3 comments
Open

slight regression in debug output #1395

simonmichael opened this issue Nov 17, 2020 · 3 comments
Labels
A-WISH Some kind of improvement request, hare-brained proposal, or plea. cli Command line parsing, options, arguments and suchlike. needs:external-task To unblock: needs completion of some task/event outside our project

Comments

@simonmichael
Copy link
Owner

simonmichael commented Nov 17, 2020 •
edited

Perhaps related to the change from pretty-show to pretty-simple. Unwanted spaces in dates and the ..:

~/src/hledger$ hledger-1.19 reg -p '2017..2018' --debug=3
requestedspan: DateSpan 2017
journalspan: DateSpan 2020-01-01 .. 2020-11-15
requestedspanclosed: DateSpan 2017
intervalspans: [ DateSpan 2017 ]
reportstart: Just 2017-01-01
reportend:  Just 2018-01-01
reportspan: DateSpan 2017
~/src/hledger$ hledger reg -p '2017..2018' --debug=3
requestedspan: DateSpan 2017
journalspan: DateSpan 2020 - 01 - 01. . 2020 - 11 - 15
requestedspanclosed: DateSpan 2017
intervalspans: [ DateSpan 2017 ]
reportstart: Just 2017 - 01 - 01
reportend:  Just 2018 - 01 - 01
reportspan: DateSpan 2017
@simonmichael simonmichael added A-BUG Something wrong, confusing or sub-standard in the software, docs, or user experience. cli Command line parsing, options, arguments and suchlike. labels Nov 17, 2020
@Xitian9
Copy link
Collaborator

Xitian9 commented Dec 14, 2020

At least some of this seems to be an instance of cdepillabout/pretty-simple#75.

@Xitian9
Copy link
Collaborator

Xitian9 commented Sep 10, 2021

What should be done about this? This is clearly an upstream issue. Perhaps we should close this and hope it's fixed there?

@simonmichael
Copy link
Owner Author

I'd definitely like this output improved in hledger, so keeping this open as a reminder.

@simonmichael simonmichael added needs:external-task To unblock: needs completion of some task/event outside our project A-WISH Some kind of improvement request, hare-brained proposal, or plea. and removed A-BUG Something wrong, confusing or sub-standard in the software, docs, or user experience. labels Jun 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-WISH Some kind of improvement request, hare-brained proposal, or plea. cli Command line parsing, options, arguments and suchlike. needs:external-task To unblock: needs completion of some task/event outside our project
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@simonmichael @Xitian9