Skip to content
Snippets Groups Projects

dev/ci: remove set -x in all tests

Merged Warren Gifford requested to merge no-x-in-tests into main

Created by: bobheadxi

IMO the utility of these are quite debatable, and they make CI output quite difficult to read. Things we can do instead:

  • Just add set -x when you're debugging something
  • Add better messages with echo
  • Move things to sg where we might be able to add a -debug mode one day

Test plan

main-dry-run: https://buildkite.com/sourcegraph/sourcegraph/builds/152078

Merge request reports

Merged by avatar (May 6, 2025 10:08pm 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