Skip to content

Commit

Permalink
Merge pull request #1080 from CircleCI-Public/life-810b
Browse files Browse the repository at this point in the history
[LIFE-810] Support additional pipeline parameters in config validation
  • Loading branch information
kira-m authored Nov 12, 2024
2 parents 0d8fde1 + b12b577 commit 08776cb
Show file tree
Hide file tree
Showing 3 changed files with 78 additions and 9 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@ The tool is deployed through a number of channels. The primary release channel i

### Homebrew

We publish the tool to [Homebrew](https://brew.sh/). The tool is [part of `homebrew-core`](https://github.com/Homebrew/homebrew-core/blob/main/Formula/circleci.rb), and therefore the maintainers of the tool are obligated to follow the guidelines for acceptable Homebrew formulae. You should [familairise yourself with the guidelines](https://docs.brew.sh/Acceptable-Formulae#we-dont-like-tools-that-upgrade-themselves) before making changes to the Homebrew deployment system.
We publish the tool to [Homebrew](https://brew.sh/). The tool is [part of `homebrew-core`](https://github.com/Homebrew/homebrew-core/blob/main/Formula/circleci.rb), and therefore the maintainers of the tool are obligated to follow the guidelines for acceptable Homebrew formulae. You should [familiarize yourself with the guidelines](https://docs.brew.sh/Acceptable-Formulae#we-dont-like-tools-that-upgrade-themselves) before making changes to the Homebrew deployment system.

The particular considerations that we make are:

Expand Down
39 changes: 31 additions & 8 deletions config/pipeline.go
Original file line number Diff line number Diff line change
Expand Up @@ -32,14 +32,37 @@ func LocalPipelineValues(parameters Parameters) Values {
}

vals := map[string]interface{}{
"pipeline.id": "00000000-0000-0000-0000-000000000001",
"pipeline.number": 1,
"pipeline.project.git_url": gitUrl,
"pipeline.project.type": projectType,
"pipeline.git.tag": git.Tag(),
"pipeline.git.branch": git.Branch(),
"pipeline.git.revision": revision,
"pipeline.git.base_revision": revision,
"pipeline.id": "00000000-0000-0000-0000-000000000001",
"pipeline.number": 1,
"pipeline.project.git_url": gitUrl,
"pipeline.project.type": projectType,
"pipeline.git.tag": git.Tag(),
"pipeline.git.branch": git.Branch(),
"pipeline.git.revision": revision,
"pipeline.git.base_revision": revision,
"pipeline.git.branch.is_default": false,
"pipeline.trigger_parameters.circleci.event_time": "2020-01-01T00:00:00Z",
"pipeline.trigger_parameters.webhook.body": "",
"pipeline.trigger_parameters.github_app.branch": "main",
"pipeline.trigger_parameters.github_app.checkout_sha": revision,
"pipeline.trigger_parameters.github_app.commit_sha": revision,
"pipeline.trigger_parameters.github_app.commit_title": "",
"pipeline.trigger_parameters.github_app.commit_message": "",
"pipeline.trigger_parameters.github_app.commit_timestamp": "2020-01-01T00:00:00Z",
"pipeline.trigger_parameters.github_app.commit_author_name": "",
"pipeline.trigger_parameters.github_app.ref": "refs/heads/master",
"pipeline.trigger_parameters.github_app.repo_name": "",
"pipeline.trigger_parameters.github_app.repo_url": "",
"pipeline.trigger_parameters.github_app.total_commits_count": 1,
"pipeline.trigger_parameters.github_app.user_avatar": "",
"pipeline.trigger_parameters.github_app.user_id": "00000000-0000-0000-0000-000000000001",
"pipeline.trigger_parameters.github_app.user_name": "",
"pipeline.trigger_parameters.github_app.user_username": "",
"pipeline.trigger_parameters.github_app.web_url": "",
"pipeline.trigger_parameters.gitlab.commit_sha": revision,
"pipeline.trigger_parameters.gitlab.default_branch": "main",
"pipeline.trigger_parameters.gitlab.x_gitlab_event_id": "00000000-0000-0000-0000-000000000001",
"pipeline.trigger_parameters.gitlab.is_fork_merge_request": false,
}

for k, v := range parameters {
Expand Down
46 changes: 46 additions & 0 deletions config/pipeline_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,29 @@ func TestLocalPipelineValues(t *testing.T) {
"pipeline.git.branch",
"pipeline.git.revision",
"pipeline.git.base_revision",
"pipeline.git.branch.is_default",
"pipeline.trigger_parameters.circleci.event_time",
"pipeline.trigger_parameters.webhook.body",
"pipeline.trigger_parameters.github_app.branch",
"pipeline.trigger_parameters.github_app.checkout_sha",
"pipeline.trigger_parameters.github_app.commit_sha",
"pipeline.trigger_parameters.github_app.commit_title",
"pipeline.trigger_parameters.github_app.commit_message",
"pipeline.trigger_parameters.github_app.commit_timestamp",
"pipeline.trigger_parameters.github_app.commit_author_name",
"pipeline.trigger_parameters.github_app.ref",
"pipeline.trigger_parameters.github_app.repo_name",
"pipeline.trigger_parameters.github_app.repo_url",
"pipeline.trigger_parameters.github_app.total_commits_count",
"pipeline.trigger_parameters.github_app.user_avatar",
"pipeline.trigger_parameters.github_app.user_id",
"pipeline.trigger_parameters.github_app.user_name",
"pipeline.trigger_parameters.github_app.user_username",
"pipeline.trigger_parameters.github_app.web_url",
"pipeline.trigger_parameters.gitlab.commit_sha",
"pipeline.trigger_parameters.gitlab.default_branch",
"pipeline.trigger_parameters.gitlab.x_gitlab_event_id",
"pipeline.trigger_parameters.gitlab.is_fork_merge_request",
},
},
{
Expand All @@ -41,6 +64,29 @@ func TestLocalPipelineValues(t *testing.T) {
"pipeline.git.base_revision",
"pipeline.parameters.foo",
"pipeline.parameters.baz",
"pipeline.git.branch.is_default",
"pipeline.trigger_parameters.circleci.event_time",
"pipeline.trigger_parameters.webhook.body",
"pipeline.trigger_parameters.github_app.branch",
"pipeline.trigger_parameters.github_app.checkout_sha",
"pipeline.trigger_parameters.github_app.commit_sha",
"pipeline.trigger_parameters.github_app.commit_title",
"pipeline.trigger_parameters.github_app.commit_message",
"pipeline.trigger_parameters.github_app.commit_timestamp",
"pipeline.trigger_parameters.github_app.commit_author_name",
"pipeline.trigger_parameters.github_app.ref",
"pipeline.trigger_parameters.github_app.repo_name",
"pipeline.trigger_parameters.github_app.repo_url",
"pipeline.trigger_parameters.github_app.total_commits_count",
"pipeline.trigger_parameters.github_app.user_avatar",
"pipeline.trigger_parameters.github_app.user_id",
"pipeline.trigger_parameters.github_app.user_name",
"pipeline.trigger_parameters.github_app.user_username",
"pipeline.trigger_parameters.github_app.web_url",
"pipeline.trigger_parameters.gitlab.commit_sha",
"pipeline.trigger_parameters.gitlab.default_branch",
"pipeline.trigger_parameters.gitlab.x_gitlab_event_id",
"pipeline.trigger_parameters.gitlab.is_fork_merge_request",
},
},
}
Expand Down

0 comments on commit 08776cb

Please sign in to comment.