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

[Feature] Removes whitespace in error message #864

Merged
merged 3 commits into from
Apr 15, 2021

Conversation

damirka
Copy link
Contributor

@damirka damirka commented Apr 14, 2021

Now we print error message this way:

Error:     --> tmp/hello-there/src/main.leo: 13:18
...

Suggested change is (look at leo:13:18 - whitespace removed):

Error:     --> tmp/hello-there/src/main.leo:13:18

Motivation

Removing whitespace would allow terminal linking in VSCode (and maybe Aleo Studio).

Test Plan

Test is changed.
This PR does not affect anything but dev experience.

@damirka damirka added the feature A new feature. label Apr 14, 2021
@damirka damirka added this to the Leo Developer Preview II milestone Apr 14, 2021
@damirka damirka self-assigned this Apr 14, 2021
@collinc97 collinc97 merged commit 7ae56f5 into master Apr 15, 2021
@collinc97 collinc97 deleted the remove-space-in-error branch April 21, 2021 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants