Skip to content

feat: remove curly braces with fmt #4529

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.

Sign up for GitHub

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

Merged
jfecher merged 13 commits into from
Mar 14, 2024
Merged

feat: remove curly braces with fmt #4529

Changes from 4 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
e3cfcbf
remove curly braces with fmt
Mar 12, 2024
de4ba4a
Merge branch 'master' into fmt_remvoes_curly_braces_when_there_is_a_s…
Andy53 Mar 12, 2024
3037c39
changed strings to chars
Mar 12, 2024
62f0f34
Update imports.rs
Andy53 Mar 12, 2024
bfc5e03
added test
Mar 12, 2024
0cf0bf6
fixed tests
Mar 12, 2024
1e2e1b8
fixing tests
Mar 12, 2024
c5035ae
fixing tests
Mar 12, 2024
487a999
fixing tests
Mar 12, 2024
e807beb
updates
Mar 14, 2024
a265e04
Update tooling/nargo_fmt/tests/expected/contract.nr
Andy53 Mar 14, 2024
9c822da
Merge branch 'master' into fmt_remvoes_curly_braces_when_there_is_a_s…
Andy53 Mar 14, 2024
89088dc
fix tests
Mar 14, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion tooling/nargo_fmt/src/rewrite/imports.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,11 @@ impl UseTree {

let mut iter = self.path.iter().peekable();
while let Some(segment) = iter.next() {
let segment_str = segment.rewrite(visitor, shape);
let mut segment_str = segment.rewrite(visitor, shape);
if segment_str.contains('{') && !segment_str.contains(',') {
let empty = "";
segment_str = segment_str.replace(['{', '}'], empty);
}
result.push_str(&segment_str);

if iter.peek().is_some() {
Expand Down