fix: automation executor updates - Githubissues

hamlet-io / executor-bash

Executor for hamlet based on bash

GNU General Public License v3.0

0 stars 4 forks source link

fix: automation executor updates #284

Closed roleyfoley closed 2 years ago

roleyfoley commented 2 years ago

Intent of Change

Bug fix (non-breaking change which fixes an issue)

Description

Fixes a variable name in the fatal messages that was hiding the role that was attempted
Skips searching for the root dir if it has already been defined

Motivation and Context

Variable name makes it easier to follow what went wrong when running a command
Skipping the root dir search means that we don't need to create the root.json file if it has already been set

How Has This Been Tested?

Tested locally

Related Changes

Prerequisite PRs:

None

Dependent PRs:

None

Consumer Actions:

None