Skip to content
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

WFCORE-5953/WFCORE-5954: Fixing several YAML extension issues #5139

Merged
merged 2 commits into from
Jun 30, 2022

Conversation

ehsavoie
Copy link
Contributor

@ehsavoie ehsavoie commented Jun 23, 2022

Two fixes for YAML Extension: empty resources and list attributes

Jira: https://issues.redhat.com/browse/WFCORE-5953
https://issues.redhat.com/browse/WFCORE-5954

…Extension.

* Adding the operation to the list of operations to be processsed.

Jira: https://issues.redhat.com/browse/WFCORE-5953

Signed-off-by: Emmanuel Hugonnet <ehugonne@redhat.com>
@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Jun 23, 2022
* When a resource is empty we need to just add it.

Jira: https://issues.redhat.com/browse/WFCORE-5954

Signed-off-by: Emmanuel Hugonnet <ehugonne@redhat.com>
@yersan
Copy link
Collaborator

yersan commented Jun 27, 2022

@ehsavoie would you mind reviewing this PR title?
It does not make any reference to the Jiras included as commits and lacks a valid description. Thanks!

@ehsavoie ehsavoie changed the title Wfcore 5953 WFCORE-5953/WFCORE-5954: Fixing several YAML extension issues Jun 27, 2022
@yersan yersan added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Jun 30, 2022
@yersan
Copy link
Collaborator

yersan commented Jun 30, 2022

@ehsavoie would you mind backporting this to 18.x too? there are other YAML issues backported there so, if finally, we want to incorporate them we can incorporate all related

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
2 participants