Skip to content

Add Quick Fix inserting shellcheck disable directive and corresponding comment #866

Open
@amurashkin17

Description

@amurashkin17

What is the problem this feature will solve?

When there is a shellcheck false positive, it is cumbersome to insert a shellcheck disable directive and a correspondent explanation comment.

IDEs often do not allow to copy the error message (for example, Eclipse). In such a case, it is necessary to type by hand the disable directive and the explanation comment.

What is the feature you are proposing to solve the problem?

It would be nice to have Quick Fix inserting shellcheck disable directive and corresponding comment, like

# disable SC2004: $/${} is unnecessary on arithmetic variables.
# shellcheck disable=SC2004 
option_value[$indx]=$2

There are some implementation hints at StackOverflow How to implement quickfix via a language server.

What alternatives have you considered?

Implementing Quick Fixes purely on Bash language client side seems less appropriate.

There is related ShellWax issue #94 Add Quick Fix inserting shellcheck disable directive and corresponding comment.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions