-
Notifications
You must be signed in to change notification settings - Fork 0
Closed
Labels
priority:lowNice to have, do when possibleNice to have, do when possiblesize:xsTrivial — less than 1 hourTrivial — less than 1 hourtype:choreMaintenance, refactoring, toolingMaintenance, refactoring, tooling
Description
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-opThe 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.rbline and its comment removed from theupdate-homebrewjob in.github/workflows/release.yml
Notes
Found during self-review of PR #58. Same pattern as: PR #56 (Nix/Scoop), PR #58 (AUR).
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
priority:lowNice to have, do when possibleNice to have, do when possiblesize:xsTrivial — less than 1 hourTrivial — less than 1 hourtype:choreMaintenance, refactoring, toolingMaintenance, refactoring, tooling