Skip to content
Snippets Groups Projects

build: Bail on error

Merged Evangelos Ribeiro Tzaras requested to merge build-set-e into master
All threads resolved!

If one is missing one of the deps (e.g. yq for android devices) we should abort early instead of continuing and producing potentially confusing errors down the road.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    Compare with previous version

  • v1.2: removed || true (which was added in v1.1) as it's not necessary (see !129 (comment 580423))

  • Arnaud Ferraris resolved all threads

    resolved all threads

  • Arnaud Ferraris approved this merge request

    approved this merge request

  • Arnaud Ferraris added 5 commits

    added 5 commits

    Compare with previous version

  • Please register or sign in to reply
    Loading