eric-maynard commented on PR #1124: URL: https://github.com/apache/polaris/pull/1124#issuecomment-2704440086
@adutra I agree there. Actually, I am hoping that when we refactor FeatureConfiguration to make it more structured, we can also resolve the issue where `QuarkusBehaviorChangeConfiguration ` and `QuarkusFeatureConfiguration` don't share a type. I also think it's a little odd that we have the logic sort of split between e.g. `PolarisFeatureConfiguration` and then `QuarkusFeatureConfiguration`, so perhaps we can fix that as well. With all that being said, I see adding a new type of config as not strictly related to these changes. Potentially, we could establish the existence of these configs here and then later refactor the `FeatureConfiguration` types -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
