Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

"Fill required fields": don't add anything in case there are no required fields or no fields at all #161

Closed
novusnota opened this issue Feb 9, 2025 · 0 comments · Fixed by #162
Labels
bug Something isn't working or doesn't seem right lsp: codeAction `textDocument/` Code Action Request

Comments

@novusnota
Copy link
Member

novusnota commented Feb 9, 2025

No required fields:

struct VM {
    exitCode: Int = 0;
    gasConsumed: Int = 0;
}

fun example() {
    // This
    VM{};

    // Should not become this
    VM{

    };
}

No fields:

message(420) TimeTo {}

fun example() {
    // This
    TimeTo{};

    // Should not become this
    TimeTo{

    };
}
@novusnota novusnota added bug Something isn't working or doesn't seem right lsp: codeAction `textDocument/` Code Action Request labels Feb 9, 2025
@novusnota novusnota changed the title "Fill required fields": don't add anything in case there are no required fields "Fill required fields": don't add anything in case there are no required fields or no fields at all Feb 9, 2025
i582 added a commit that referenced this issue Feb 9, 2025
… there are no such fields, or no fields to fill in at all

Fixes #161
@i582 i582 closed this as completed in #162 Feb 9, 2025
i582 added a commit that referenced this issue Feb 9, 2025
… there are no such fields, or no fields to fill in at all (#162)

Fixes #161
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working or doesn't seem right lsp: codeAction `textDocument/` Code Action Request
Projects
None yet
1 participant