Skip to content

Commit 274befd

Browse files
Merge pull request #47 from mineiros-io/update-ci-comment
Update CI environment variables comment in Makefile to use proper wording.
2 parents 808ebb3 + 224501c commit 274befd

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

Makefile

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,16 @@ BUILD_TOOLS_VERSION ?= v0.5.4
55
BUILD_TOOLS_DOCKER_REPO ?= mineiros/build-tools
66
BUILD_TOOLS_DOCKER_IMAGE ?= ${BUILD_TOOLS_DOCKER_REPO}:${BUILD_TOOLS_VERSION}
77

8-
# if running in CI (e.g. Semaphore CI)
9-
# https://docs.semaphoreci.com/ci-cd-environment/environment-variables/#ci
108
#
11-
# to disable TF_IN_AUTOMATION in CI set it to empty
9+
# Some CI providers such as GitHub Actions, CircleCI, and TravisCI are setting
10+
# the CI environment variable to a non-empty value by default to indicate that
11+
# the current workflow is running in a Continuous Integration environment.
12+
#
13+
# If TF_IN_AUTOMATION is set to any non-empty value, Terraform adjusts its
14+
# output to avoid suggesting specific commands to run next.
1215
# https://www.terraform.io/docs/commands/environment-variables.html#tf_in_automation
1316
#
14-
# we are using GNU style quiet commands to disable set V to non-empty e.g. V=1
17+
# We are using GNU style quiet commands to disable set V to non-empty e.g. V=1
1518
# https://www.gnu.org/software/automake/manual/html_node/Debugging-Make-Rules.html
1619
#
1720
ifdef CI

0 commit comments

Comments
 (0)