lexer: add Hint to Error, which is expanded as "%s" instead of "%q" #12

Merged

amery merged 2 commits from pr-amery-error into main 2 years ago

amery commented 2 years ago

Owner

There is no content yet.

amery added 1 commit 2 years ago

eb2e4ef83c lexer: add Hint to Error, which is expanded as "%s" instead of "%q"

Signed-off-by: Alejandro Mery <[email protected]>

amery requested review from karasz 2 years ago

amery added 2 commits 2 years ago

4aa6233e4f lexer: refactor Error.Error()'s prefix generator

Signed-off-by: Alejandro Mery <[email protected]>

2b5dcec64d lexer: add Hint to Error, which is expanded as "%s" instead of "%q"

Signed-off-by: Alejandro Mery <[email protected]>

amery commented 2 years ago

Poster

Owner

refactored to reduce complexity

amery added 2 commits 2 years ago

1d7ee69ab7 lexer: refactor Error.Error()'s prefix generator

Signed-off-by: Alejandro Mery <[email protected]>

ce75299e74 lexer: add Hint to Error, which is expanded as "%s" instead of "%q"

Signed-off-by: Alejandro Mery <[email protected]>

karasz approved these changes 2 years ago

karasz left a comment

Owner

LGTM

amery merged commit 5f81eb0ea5 into main 2 years ago

amery deleted branch pr-amery-error 2 years ago

amery referenced this issue from a commit 2 years ago

Merge pull request 'lexer: add Hint to Error, which is expanded as "%s" instead of "%q"' (#12)

Reviewers

karasz approved these changes 2 years ago

The pull request has been merged as 5f81eb0ea5.

Sign in to join this conversation.

No Label

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.