hamlet-io / executor-bash

Executor for hamlet based on bash
GNU General Public License v3.0
0 stars 4 forks source link

refactor: set credentials #343

Closed ml019 closed 2 years ago

ml019 commented 2 years ago

Intent of Change

Description

Remove a redundant condition around the definition of local_aws_auth_role.

Motivation and Context

The conditional check uses what appears to be a mis-spelt variable name and hence is always treated as true.

Later code relies on local_aws_auth_role being defined so if the conditional ever returned false, later code would fail.

How Has This Been Tested?

Related Changes

Prerequisite PRs:

Dependent PRs:

Consumer Actions: