slaesvuo/hledger
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test: errors: doc updates

Browse Source
This commit is contained in:
Simon Michael 2022-03-20 07:21:31 -10:00
parent 5281c5d647
commit 4eeb646912
1 changed files with 19 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
hledger/test/errors/README.md
View File

@ -26,24 +26,25 @@ brings immediate practical benefits.
Here is the current status Here is the current status
(hledger 1.25, flycheck + [PR#10][flycheck-hledger-10]): (hledger 1.25, flycheck + [PR#10][flycheck-hledger-10]):
| | format | accurate line(s) | accurate column(s) | visual | flycheck detects | flycheck region | | | format | accurate line(s) | accurate column(s) | visual | flycheck detects | flycheck region |
|--------------------------|----------|------------------|--------------------|--------|------------------|-----------------| |--------------------------|---------|------------------|--------------------|--------|------------------|-----------------|
| parseable | format 1 | Y | Y | YY | Y | Y | | parseable | format1 | Y | Y | YY | Y | Y |
| parseable-dates | format 1 | Y | Y | YY | Y | Y | | parseable-dates | format1 | Y | Y | YY | Y | Y |
| parseable-regexps | format 1 | Y | Y | YY | Y | Y | | parseable-regexps | format1 | Y | Y | YY | Y | Y |
| balanced | | Y | - | Y | Y | | | balanced | | Y | - | Y | Y | |
| balancednoautoconversion | | Y | - | Y | Y | | | balancednoautoconversion | | Y | - | Y | Y | |
| assertions | | Y | | Y | Y | Y | | assertions | | Y | | Y | Y | Y |
| accounts | format 2 | | | Y | Y | | | accounts | format2 | | | Y | Y | |
| commodities | format 2 | | | Y | Y | | | commodities | format2 | | | Y | Y | |
| payees | format 2 | | | Y | Y | Y | | payees | format2 | | | Y | Y | Y |
| ordereddates | format 2 | | | Y | Y | Y | | ordereddates | format2 | | | Y | Y | Y |
| uniqueleafnames | | | | Y | Y | | | uniqueleafnames | | | | Y | Y | |
Key: Key:
- format: the error message follows a standard format - format: the error message follows a standard format
(format 1: location on first line, megaparsec-like. (format1: location on first line, megaparsec-like.
format 2: summary on first line, location on second line, rustc-like.) format2: summary on first line, location on second line, rustc-like.
std: new standard format)
- accurate line - the optimal line(s) is(are) selected - accurate line - the optimal line(s) is(are) selected
- accurate column - the optimal column(s) is(are) selected - accurate column - the optimal column(s) is(are) selected
- visual - the CLI error message shows a relevant excerpt (Y), ideally with the error highlighted (YY) - visual - the CLI error message shows a relevant excerpt (Y), ideally with the error highlighted (YY)
@ -55,7 +56,7 @@ Key:
- [ ] phase 1: update flycheck to detect journal errors of current hledger release (and keep a branch updated to detect errors of latest hledger master) - [ ] phase 1: update flycheck to detect journal errors of current hledger release (and keep a branch updated to detect errors of latest hledger master)
- [x] phase 2: survey/document current journal errors - [x] phase 2: survey/document current journal errors
- [x] phase 3: pick a new standard format - [x] phase 3: pick a new standard format
- [ ] phase 4: implement consistent format for all - [ ] phase 4: implement standard format for all
- [ ] phase 5: implement accurate lines for all - [ ] phase 5: implement accurate lines for all
- [ ] phase 6: implement accurate columns for all - [ ] phase 6: implement accurate columns for all
- [ ] phase 7: implement useful highlighted excerpts for all - [ ] phase 7: implement useful highlighted excerpts for all
@ -201,7 +202,7 @@ seen in "a:c" in transaction at: /Users/simon/src/hledger/hledger/test/errors/./
## New error format ## New error format
The preferred standard error format for now is the following, The preferred standard error format for now is the following,
similar to the one megaparsec gives us similar to the one megaparsec gives us
and probably the easiest to implement consistently: and probably the easiest to implement consistently:
@ -209,7 +210,7 @@ and probably the easiest to implement consistently:
Error: FILE:LOCATION: Error: FILE:LOCATION:
EXCERPT EXCERPT
SUMMARY SUMMARY
[DETAILS} [DETAILS]
``` ```
Other format notes (see also [#1436][]): Other format notes (see also [#1436][]):
@ -249,4 +250,3 @@ Questions:
- allow long, much-wider-than-80-char first lines or not ? - allow long, much-wider-than-80-char first lines or not ?
- dynamic or static summary ? - dynamic or static summary ?
- error ids/explanations needed ? local and/or web based ? easily editable ? document old hledger versions ? - error ids/explanations needed ? local and/or web based ? easily editable ? document old hledger versions ?