Skip to content

Commit de9daa8

Browse files
committed
simplify parameter usage
1 parent 05bafcd commit de9daa8

File tree

2 files changed

+2
-11
lines changed

2 files changed

+2
-11
lines changed

.circleci/src/pipeline/@pipeline.yml

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -119,16 +119,12 @@ commands:
119119
# This command inserts SHOULD_PERSIST_ARTIFACTS into BASH_ENV. This way, we can define the variable in one place and use it in multiple steps.
120120
# Run this command in a job before you want to use the SHOULD_PERSIST_ARTIFACTS variable.
121121
setup_should_persist_artifacts:
122-
parameters:
123-
force-persist-artifacts:
124-
type: boolean
125-
default: false
126122
steps:
127123
- run:
128124
name: Set environment variable to determine whether or not to persist artifacts
129125
command: |
130126
echo "Setting SHOULD_PERSIST_ARTIFACTS variable"
131-
if [[ "<< parameters.force-persist-artifacts >>" == "true" ]]; then
127+
if [[ "<< pipeline.parameters.force-persist-artifacts >>" == "true" ]]; then
132128
echo 'export SHOULD_PERSIST_ARTIFACTS=true' >> "$BASH_ENV"
133129
else
134130
echo 'if ! [[ "$CIRCLE_BRANCH" != "develop" && "$CIRCLE_BRANCH" != "release/"* ]]; then
@@ -2483,18 +2479,14 @@ jobs:
24832479
resource_class:
24842480
type: string
24852481
default: medium
2486-
force-persist-artifacts:
2487-
type: boolean
2488-
default: false
24892482
resource_class: << parameters.resource_class >>
24902483
steps:
24912484
- maybe_skip_binary_jobs
24922485
- restore_cached_workspace
24932486
- run:
24942487
name: Check pipeline info
24952488
command: cat ~/triggered_pipeline.json
2496-
- setup_should_persist_artifacts:
2497-
force-persist-artifacts: << parameters.force-persist-artifacts >>
2489+
- setup_should_persist_artifacts
24982490
- run:
24992491
name: Download binary artifacts
25002492
command: |

.circleci/src/pipeline/workflows/pull-request.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -326,7 +326,6 @@ jobs:
326326
requires:
327327
- create-and-trigger-packaging-artifacts
328328
- get-published-artifacts:
329-
force-persist-artifacts: << pipeline.parameters.force-persist-artifacts >>
330329
context:
331330
- publish-binary
332331
- test-runner:commit-status-checks

0 commit comments

Comments
 (0)