Skip to content

Add spec.files for Condition and Target of kind yaml#650

Merged
olblak merged 53 commits into
updatecli:mainfrom
lemeurherve:is-435-yaml-files
Feb 1, 2023
Merged

Add spec.files for Condition and Target of kind yaml#650
olblak merged 53 commits into
updatecli:mainfrom
lemeurherve:is-435-yaml-files

Conversation

@lemeurherve

@lemeurherve lemeurherve commented Apr 14, 2022

Copy link
Copy Markdown
Member

feat(yaml): implement spec.files for kind: yaml

Follow-up of #490
Ref #435

Allow to specify more than one file as target for kind: yaml

Like what have been done for kind: file, the possibility to specify multiple files is only possible in the targets stage.

Test

To test this pull request, you can run the following commands:

cp <to_package_directory>
go test

Additional Information

Tradeoff

Potential improvement

@lemeurherve lemeurherve added the enhancement New feature or request label Apr 14, 2022
@lemeurherve lemeurherve marked this pull request as ready for review April 14, 2022 16:14
@lemeurherve lemeurherve requested review from dduportal and olblak April 14, 2022 16:14
@lemeurherve lemeurherve added the resource-yaml Resource of kind YAML label Apr 14, 2022
Comment thread examples/updatecli.generic/yaml-files.yaml Outdated
Comment thread pkg/plugins/resources/yaml/condition.go Outdated
Comment thread pkg/plugins/resources/yaml/main.go Outdated
Comment thread pkg/plugins/resources/yaml/main.go Outdated
Comment thread pkg/plugins/resources/yaml/main.go Outdated
Comment thread pkg/plugins/resources/yaml/main.go Outdated
Comment thread pkg/plugins/resources/yaml/source.go Outdated
Comment thread pkg/plugins/resources/yaml/source.go Outdated
@lemeurherve lemeurherve marked this pull request as draft June 5, 2022 14:33
@lemeurherve lemeurherve marked this pull request as ready for review June 21, 2022 13:09
@lemeurherve lemeurherve changed the title feat(yaml): implement spec.files for Target of kind yaml feat(yaml): implement spec.files for Conditions and Target of kind yaml Jan 29, 2023
@lemeurherve lemeurherve added the condition Modify condition resource label Jan 29, 2023
@lemeurherve lemeurherve changed the title feat(yaml): implement spec.files for Conditions and Target of kind yaml feat(yaml): implement spec.files for Condition and Target of kind yaml Jan 29, 2023
dduportal
dduportal previously approved these changes Jan 29, 2023
olblak
olblak previously approved these changes Feb 1, 2023
Comment thread pkg/plugins/resources/yaml/main.go Outdated
@olblak olblak enabled auto-merge (squash) February 1, 2023 14:02
@olblak olblak changed the title feat(yaml): implement spec.files for Condition and Target of kind yaml Add spec.files for Condition and Target of kind yaml Feb 1, 2023
@olblak olblak disabled auto-merge February 1, 2023 14:29
@olblak olblak merged commit 58c4303 into updatecli:main Feb 1, 2023
@olblak

olblak commented Feb 1, 2023

Copy link
Copy Markdown
Member

Thanks @hervelemeur for your patience

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

condition Modify condition resource enhancement New feature or request resource-yaml Resource of kind YAML target Related to updatecli target

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants