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

leo error reporting after build #28327

Open
wants to merge 3 commits into
base: mainnet
Choose a base branch
from

Conversation

ungaro
Copy link
Contributor

@ungaro ungaro commented Aug 18, 2024

Motivation

When developing complex Leo programs, it's not uncommon to encounter multiple compilation errors. This feature, inspired by Rust's error reporting system, provides developers with a clear overview of the number of errors and warnings in their code. Having a precise count of issues serves as a motivational tool, allowing developers to track their progress as they address and resolve bugs. The satisfaction of seeing the error count decrease provides positive reinforcement throughout the debugging process, ultimately enhancing the overall development experience and potentially improving code quality.

Test Plan

Run a leo program that generates errors and warnings.

Here is an example of what it looks like:
leo_error_reporting

@ungaro
Copy link
Contributor Author

ungaro commented Aug 18, 2024

would love to have your review @d0cd

compiler/passes/src/type_checking/mod.rs Outdated Show resolved Hide resolved
compiler/passes/src/type_checking/mod.rs Outdated Show resolved Hide resolved
@ungaro ungaro requested a review from d0cd August 30, 2024 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants