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.

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

Add optional code-frame to show error location #114

Merged
merged 6 commits into from Jun 8, 2020
Merged

Add optional code-frame to show error location #114

merged 6 commits into from Jun 8, 2020

Conversation

@marvinhagemeister
Copy link
Member

marvinhagemeister commented Jun 7, 2020

This PR adds a code-frame to directly show where the error occurred in relation to the original source code. It's optional and will only be present if the error was thrown by user code.

Colors:

Screenshot from 2020-06-08 00-19-25

No colors:

Screenshot from 2020-06-08 00-24-44

output.js Outdated Show resolved Hide resolved
output.js Outdated Show resolved Hide resolved
@phihag
Copy link
Collaborator

phihag commented Jun 8, 2020

Nice! I added two tiny nitpicks, but otherwise LGTM.

@marvinhagemeister marvinhagemeister force-pushed the code-frame branch from 03a6453 to 4c5f837 Jun 8, 2020
@marvinhagemeister marvinhagemeister force-pushed the code-frame branch from 4c5f837 to 7ddedfe Jun 8, 2020
@phihag phihag merged commit 55de6cc into master Jun 8, 2020
4 checks passed
4 checks passed
build (10.x)
Details
build (11.x)
Details
build (12.x)
Details
build (14.x)
Details
@phihag phihag deleted the code-frame branch Jun 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.