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

Lint pass that warns on try expressions without raise expressions or error-propagating calls #66

Closed
alexrp opened this issue Mar 31, 2023 · 0 comments
Labels
area: analysis Issues related to language analyses. area: tests Issues related to the test suite. type: feature Issues that are classified as feature requests.
Milestone

Comments

@alexrp
Copy link
Sponsor Member

alexrp commented Mar 31, 2023

This try is pointless and misleading:

try foo() catch { _ => 42 };

We should warn on this.

@alexrp alexrp added state: approved Feature requests and housekeeping tasks that have been approved. type: feature Issues that are classified as feature requests. area: analysis Issues related to language analyses. area: tests Issues related to the test suite. labels Mar 31, 2023
@alexrp alexrp added this to the v1.0 milestone Mar 31, 2023
@alexrp alexrp self-assigned this Mar 31, 2023
@alexrp alexrp closed this as completed in 7b348f7 Mar 31, 2023
@alexrp alexrp removed the state: approved Feature requests and housekeeping tasks that have been approved. label Mar 31, 2023
@alexrp alexrp removed their assignment Jan 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: analysis Issues related to language analyses. area: tests Issues related to the test suite. type: feature Issues that are classified as feature requests.
Development

No branches or pull requests

1 participant