jamesnetherton commented on a change in pull request #1258:
URL: https://github.com/apache/camel-quarkus/pull/1258#discussion_r430182941
##########
File path: .github/test-categories.yaml
##########
@@ -32,7 +32,7 @@ main:
- main-xml-jaxb
database:
- couchdb
- - debezium-postgres
+ - debezium-common
Review comment:
Do we need the `-common` suffix? If not, lets simplify to `debezium`.
##########
File path: integration-tests/debezium-common/pom.xml
##########
@@ -28,9 +28,9 @@
<relativePath>../pom.xml</relativePath>
</parent>
- <artifactId>camel-quarkus-integration-test-debezium-postgres</artifactId>
- <name>Camel Quarkus :: Integration Tests :: Debezium PostgresSQL
Connector</name>
- <description>Integration tests for Camel Quarkus Debezium PostgresSQL
Connector extension</description>
+ <artifactId>camel-quarkus-integration-test-debezium-common</artifactId>
+ <name>Camel Quarkus :: Integration Tests :: Debezium Connectors</name>
+ <description>Integration tests for Camel Quarkus Debezium PostgresSQL,
MySQL Connector extension</description>
Review comment:
Better to just have `Integration tests for Camel Quarkus Debezium
Extensions`, else the description will get ever larger once support for other
stuff is added.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
