Include filename in parse error messages #175

New Issue

2018-08-22T20:32:40+02:00

pbrisbin commented

2018-08-22 20:32:40 +02:00

(Migrated from github.com)

When one runs brittany on multiple files and one of them errors, you can't tell which file had the error:

% brittany --write-mode inplace **/*.hs
ERROR: encountered unknown syntactical constructs:
HsSpliceE{}
HsSpliceE{}
ERROR: encountered unknown syntactical constructs:
HsSpliceE{}

When the file list is very large, it becomes impossible to track things down.

This is probably because the error message format was decided before --write-mode inplace existed, so operating over multiple files didn't make much sense. Now that we have that though, I would like to request that the filename be included in these errors.

When one runs brittany on multiple files and one of them errors, you can't tell which file had the error: ```console % brittany --write-mode inplace **/*.hs ERROR: encountered unknown syntactical constructs: HsSpliceE{} HsSpliceE{} ERROR: encountered unknown syntactical constructs: HsSpliceE{} ``` When the file list is _very_ large, it becomes impossible to track things down. This is probably because the error message format was decided before `--write-mode inplace` existed, so operating over multiple files didn't make much sense. Now that we have that though, I would like to request that the filename be included in these errors.

👍 2

Sign in to join this conversation.

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: hexagoxel/brittany#175