syndesisio / syndesis-extensions

Collection of Syndesis extensions
Apache License 2.0
31 stars 73 forks source link

Upgrade syndesis version and make API changes to Step #68

Closed cunningt closed 4 years ago

cunningt commented 6 years ago

I'm not sure what we want to do here - the following commit changed the Step API in a way that requires changes from the syndesis-extensions actions :

https://github.com/syndesisio/syndesis/commit/91e424a3c18d105efd6915612b644c6b51c0954e#diff-1ce34ed4cb8daaadfd2ab43eef0d8219L27

I'd like a branch of syndesis-extensions that can be compiled against syndesis 1.4 - can we create a 1.4.x branch of syndesis-extensions and incorporate these changes within?

zregvart commented 6 years ago

@syndesisio/backend any thoughts here?

lburgazzoli commented 6 years ago

The idea was to use the first 7.x tag for compatibility reasons, people that need specific version need to set in in theirs build.

I think the best would be to remove the generic diamond in extension APIs for fuse 7.x and introduce it on 8.x series.

squakez commented 4 years ago

This was open since a very long time now. We have already included that change in another work done. Thanks for the PR anyway :)