squakez commented on code in PR #3912: URL: https://github.com/apache/camel-k/pull/3912#discussion_r1054093419
########## pkg/apis/camel/v1/maven_types.go: ########## @@ -96,4 +96,9 @@ type Server struct { Configuration Properties `xml:"configuration,omitempty" json:"configuration,omitempty"` } -type Properties map[string]string Review Comment: This change here looks to be breaking compatibility in the API. In order to be performed we should either have a backward compatible field or deprecate the previous one and make it coexist for a couple of versions with the newer. -- 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