-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: artifact support pattern and merge_multiple #139
Conversation
Terraform plan for sandbox Changes to Outputs.Changes to Outputs:
+ file = <<-EOT
|------------------|
| I am an artifact |
|------------------|
EOT
You can apply this plan to save these new output values to the Terraform
state, without changing any real infrastructure. 📝 Plan generated in .github/workflows/_test-tf.yaml #218 |
Terraform plan in tests/terraform/s3 Plan: 2 to add, 0 to change, 0 to destroy. Changes to Outputs.Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
+ create
Terraform will perform the following actions:
# random_id.this will be created
+ resource "random_id" "this" {
+ b64_std = (known after apply)
+ b64_url = (known after apply)
+ byte_length = 4
+ dec = (known after apply)
+ hex = (known after apply)
+ id = (known after apply)
}
# random_pet.this will be created
+ resource "random_pet" "this" {
+ id = (known after apply)
+ length = 2
+ separator = "-"
}
Plan: 2 to add, 0 to change, 0 to destroy.
Changes to Outputs:
+ random_id = (known after apply)
+ random_pet = (known after apply) 📝 Plan generated in .github/workflows/_test-tf.yaml #218 |
e4f901b
to
6a68569
Compare
Terraform plan for sandbox Plan: 2 to add, 0 to change, 0 to destroy. Changes to Outputs.Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
+ create
Terraform will perform the following actions:
# random_id.this will be created
+ resource "random_id" "this" {
+ b64_std = (known after apply)
+ b64_url = (known after apply)
+ byte_length = 4
+ dec = (known after apply)
+ hex = (known after apply)
+ id = (known after apply)
}
# random_pet.this will be created
+ resource "random_pet" "this" {
+ id = (known after apply)
+ length = 2
+ separator = "-"
}
Plan: 2 to add, 0 to change, 0 to destroy.
Changes to Outputs:
+ random_id = (known after apply)
+ random_pet = (known after apply) 📝 Plan generated in .github/workflows/_test-tf.yaml #218 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rerun the docs script so the default value is correct
This reverts commit d826ad2.
Description
Add support for artifact features described here https://github.com/actions/download-artifact/tree/fa0a91b85d4f404e444e00e005971372dc801d16/?tab=readme-ov-file#inputs
Introduced inputs are pattern and merge_multiple
Motivation and Context
Requirements in case multiple artifacts needs to be pushed at different steps of the pipeline
Breaking Changes
No
How Has This Been Tested?
pre-commit run -a
on my pull requestmake gen_docs_run
on my pull request