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

nargo fmt should remove unnecessary {} in nested imports #4558

Open
jfecher opened this issue Mar 14, 2024 · 0 comments
Open

nargo fmt should remove unnecessary {} in nested imports #4558

jfecher opened this issue Mar 14, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@jfecher
Copy link
Contributor

jfecher commented Mar 14, 2024

Aim

See #4529 (comment) for an example which is not caught by the current formatter:

use foo::{
    bar::baz,
    qux::{quux},  // unnecessary but not removed
};

Expected Behavior

The formatter should remove {} on single imports, even if the import is nested

Bug

The formatter does not do so

To Reproduce

Project Impact

None

Impact Context

No response

Workaround

None

Workaround Description

No response

Additional Context

No response

Installation Method

None

Nargo Version

No response

NoirJS Version

No response

Would you like to submit a PR for this Issue?

None

Support Needs

No response

@jfecher jfecher added the bug Something isn't working label Mar 14, 2024
@jfecher jfecher changed the title nargo fmt should remove unnecessary { .. } in nested imports nargo fmt should remove unnecessary {} in nested imports Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 📋 Backlog
Development

No branches or pull requests

1 participant