parodos-dev / serverless-workflow-examples

Apache License 2.0
0 stars 13 forks source link

tekton workflow example #34

Closed ydayagi closed 3 months ago

ydayagi commented 3 months ago

FLPATH-938 https://issues.redhat.com/browse/FLPATH-938

ydayagi commented 3 months ago

@dmartinol @masayag what say u? approve or not? do you want to merge your PRs that provide a stable template and then I'll rework based on that? or should we merge this one cause it's working fine and does nothing bad and later change it?

dmartinol commented 3 months ago

@dmartinol @masayag what say u? approve or not? do you want to merge your PRs that provide a stable template and then I'll rework based on that? or should we merge this one cause it's working fine and does nothing bad and later change it?

Sorry but I have no PRs in mind for this repo. I hope you can accept the latest suggestion that I provided. To me it's not acceptable to push changes that are not updated with the latest versions, at least now that we are aware of that. Maybe @masayag has a different opinion.

ydayagi commented 3 months ago

@dmartinol @masayag what say u? approve or not? do you want to merge your PRs that provide a stable template and then I'll rework based on that? or should we merge this one cause it's working fine and does nothing bad and later change it?

Sorry but I have no PRs in mind for this repo. I hope you can accept the latest suggestion that I provided. To me it's not acceptable to push changes that are not updated with the latest versions, at least now that we are aware of that. Maybe @masayag has a different opinion.

I updated the versions section to include newest versions. using the parent pom did not work.