From 697cf0ccee6c79c121b79f5f3df170f581decd86 Mon Sep 17 00:00:00 2001 From: Cole McAnelly <76577549+colemcanelly@users.noreply.github.com> Date: Thu, 14 May 2026 11:22:49 -0500 Subject: [PATCH] Revert: "languages: Improve function parameter highlighting for Bash" (#52245) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Context Closes https://github.com/zed-industries/zed/issues/52242 This PR reverts a change that highlights shell script arguments as variables rather than strings. Because of how shell scripts work, all parameters are strings by default and are regularly represented as such. The original PR (https://github.com/zed-industries/zed/pull/48067) aimed to improve highlighting by capturing all arguments as `variable.parameter`s, but this doesn't align with the idiomatic usage of shell scripts. | Current Behavior | Expected Behavior | |---|---| | Image | Image | | Text arguments are highlighted as variable | Text arguments are highlighted as "string" | ## Self-Review Checklist - [x] I've reviewed my own diff for quality, security, and reliability - [x] Unsafe blocks (if any) have justifying comments - [x] The content is consistent with the [UI/UX checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist) - [ ] Tests cover the new/changed behavior - [ ] Performance impact has been considered and is acceptable Release Notes: - N/A or Added/Fixed/Improved ... --- crates/grammars/src/bash/highlights.scm | 7 ------- 1 file changed, 7 deletions(-) diff --git a/crates/grammars/src/bash/highlights.scm b/crates/grammars/src/bash/highlights.scm index c67b34e92c9..99401a59435 100644 --- a/crates/grammars/src/bash/highlights.scm +++ b/crates/grammars/src/bash/highlights.scm @@ -49,13 +49,6 @@ (command_name (word) @function) -(command - argument: [ - (word) @variable.parameter - (_ - (word) @variable.parameter) - ]) - [ (file_descriptor) (number)