squakez commented on code in PR #4456:
URL: https://github.com/apache/camel-k/pull/4456#discussion_r1219059623


##########
resources/camel-catalog-2.16.0-SNAPSHOT.yaml:
##########
@@ -38,6 +38,14 @@ spec:
     - groupId: org.apache.camel.k
       artifactId: camel-k-runtime
     capabilities:

Review Comment:
   I think we better not mix concepts. Capabilities are one specific thing. IMO 
we should introduce a new concept, `Traits` where we include the logic 
specifically required for traits. Although at the beginning they will contain 
the same information (basically a list of dependencies), it's very likely they 
will evolve differently. I think we're likely going to leverage the same field 
later on in order to include properties and any other trait specific logic 
which does not necessarily belong to Capabilities.



-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to