Skip to content

Removal of conditions which are always evaluated either to true or false

Alexander Kutelev requested to merge kutelev/gitlab-runner:suspicious-condition into master

What does this MR do?

  1. Removes suspicious condition which is always evaluated to true. executor is compared against nil at a location where executor can never be nil.
  2. Removes comparisons of authConfigs with nil. authConfigs can never be nil.

Why was this MR needed?

General cleanup.

Are there points in the code the reviewer needs to double check?

-

What are the relevant issue numbers?

-

Edited by Alexander Kutelev

Merge request reports