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 1 year ago

amery commented 1 year ago

Owner

There is no content yet.

amery added 1 commit 1 year ago

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

Signed-off-by: Alejandro Mery <amery@jpi.io>

amery requested review from karasz 1 year ago

amery added 2 commits 1 year ago

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

Signed-off-by: Alejandro Mery <amery@jpi.io>

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

Signed-off-by: Alejandro Mery <amery@jpi.io>

amery commented 1 year ago

Poster

Owner

refactored to reduce complexity

amery added 2 commits 1 year ago

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

Signed-off-by: Alejandro Mery <amery@jpi.io>

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

Signed-off-by: Alejandro Mery <amery@jpi.io>

karasz approved these changes 1 year ago

karasz left a comment

Owner

LGTM

amery merged commit 5f81eb0ea5 into main 1 year ago

amery deleted branch pr-amery-error 1 year ago

amery referenced this issue from a commit 1 year ago

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

Reviewers

karasz approved these changes 1 year 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.