Skip to content

fix: add inline shellcheck disable for SC2012 in wrappers #334

Description

@gtrabanco

Issue

Bunx/npx wrappers have file-level # shellcheck disable=SC2012 at line 3, but shellcheck only respects inline disables - SC2012 fires on ls -1 lines.

Root cause

File-level disable directive does not suppress line-level warnings. Need inline # shellcheck disable=SC2012 before each ls -1 line.

Occurrences per wrapper: lines 84, 91, 103.

Scope

In scope: inline disable before each ls -1 (6 lines total). Out of scope: adding shellcheck to CI, replacing ls -1.

Acceptance criteria

  • Both wrappers have inline disable before each ls -1 line
  • static_analysis passes - zero warnings
  • lint passes

Effort

XS - 6 lines.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions