Hello, comments inline:
On Sat, Jan 15, 2022, at 06:37, David Jencks wrote: > I noticed a few things working on the RI info for camel-kafka-connector. > > - the compatibility matrices are thoroughly out of date, e.g. > https://camel.apache.org/camel-kafka-connector/0.11.0/user-guide/camel-compatibility-matrix.html > > <https://camel.apache.org/camel-kafka-connector/0.11.0/user-guide/camel-compatibility-matrix.html> Yep the compatibility matrix page needs some love... a column mentioning kamelet catalog version needs to be added and probably we can remove some old rows? willing to help on this on too? :) > > - All other camel subprojects use e.g. 2.5.x as the Antora component version, > but c-k-c is using 0.11.0. Especially since it’s LTS I think we should > change it to 0.11.x so when 0.11.1 comes out the version still makes sense, > as well as being consistent with the rest of the site. I’m setting the 1.0.x > branch up to say 1.0.x as the Antora version. +1 I agree > > - archetype-dataformat-connector has camel-version 3.6.0, rather out of date. What do you mean here? > > - the maven versions in 1.0.x branch is 1.1.0-SNAPSHOT and for 0.11.x > 0.12.0-SNAPSHOT. I think these should increment the micro version, not minor > version? In theory you are right, in practice for convenience and based on how maven release plugin works and how we do releases, is more convenient to leave the version set as next release version even in the single releases branches... hoping to remember that once a new minor release of a release branch is needed. I admit that might be we are just being lazy and there is a reasonably hassle free way of handling this better? > > - Is 1.0.x LTS? Yes it is, as it is 0.11.0 > > - I guess it would make sense to change > > Camel Kafka Connector allows you to use all Camel components > <applewebdata://26B8BDA8-8AF9-4D43-86E9-44E7AD9124B6/components/3.14.x/index.html> > as Kafka Connect <http://kafka.apache.org/documentation/#connect> connectors. > to > Camel Kafka Connector allows you to use all Kamelets as Kafka Connect > <http://kafka.apache.org/documentation/#connect> connectors.* +1 I agree > > As part of the RI effort there’s a preview for c-k-c at > https://pr-747--camel.netlify.app/camel-kafka-connector/next/index.html > > David Jencks
