Skip to content

Remove dead sed strip in update-homebrew CI job #59

@vmvarela

Description

@vmvarela

Description

The update-homebrew job in release.yml has the same dead sed strip that was fixed for AUR (PR #58), Nix, and Scoop (PR #56):

# Strip leading whitespace from heredoc (was indented for readability)
sed -i 's/^          //' sql-pipe.rb   # <-- no-op

The heredoc uses a quoted delimiter (<<'FORMULA'), so YAML block-scalar indentation stripping never runs inside it, and the sed pattern matches nothing. Safe to remove.

Acceptance Criteria

  • Dead sed -i 's/^ //' sql-pipe.rb line and its comment removed from the update-homebrew job in .github/workflows/release.yml

Notes

Found during self-review of PR #58. Same pattern as: PR #56 (Nix/Scoop), PR #58 (AUR).

Metadata

Metadata

Assignees

No one assigned

    Labels

    priority:lowNice to have, do when possiblesize:xsTrivial — less than 1 hourtype:choreMaintenance, refactoring, tooling

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions