lsergio commented on issue #5838:
URL: https://github.com/apache/camel-k/issues/5838#issuecomment-2358933996
Hi @squakez. I tried working on this for the last few days, but it would
definitely require more effort and a deeper understanding of the code than I
have.
I see that
[BuildConfiguration](https://github.com/apache/camel-k/blob/7aa5b550e82d7ae27022a050d94ff3338872efc3/pkg/apis/camel/v1/common_types.go#L41)
is being used at the IntegrationPlatform spec and at the Build spec. In the
Build it's used as a Build property and also at the Task level.
Also, as I have mentioned before, this has a lot in common with the
BuilderTrait.
I tried using specific structs for each location where BuildConfiguration is
currently used, keeping only what makes sense at each location, but there was a
lot of impacts in the code, and that's where I think someone with a deeper
understanding of the project might handle it better than me.
I filed it as bug, but maybe this has to change to some refactoring work for
2.6.0+, as the 2.5.0 date is very close.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
