juergbi commented on PR #2102: URL: https://github.com/apache/buildstream/pull/2102#issuecomment-3824711498
The proposed behavior seems reasonable. Looking at the documentation, the current `none` behavior might not have been intentional. It is technically still a breaking change from the CLI/config perspective but it may be acceptable for 2.7 given the default user config update which mitigates the change (and it won't break project config or element definitions). I'd like to see at least minimal test coverage, though (could be a current test that is slightly expanded). -- 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]
