Skip to content
Snippets Groups Projects

ci: detect bash scripts with shebang

Merged Warren Gifford requested to merge devx/fix-diff-shell-detection into main

Created by: jhchabran

Fixes #38754 (closed)

One could argue that all shell scripts should end in .sh but it's actually quite common in the wild to just use the shebang, so we'd better address that case for the sake of correctness.

Test plan

Ran the generator locally, with a commit including changes to /dev/zoekt/update which is a shell script, but without the extension.

Merge request reports

Merged by avatar (Apr 24, 2025 6:03am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading