abderrahim commented on PR #1877: URL: https://github.com/apache/buildstream/pull/1877#issuecomment-1942152625
> This was supposed to be BuildStream 2 behaviour. BuildStream 1 warned about this and stated it's going to start failing. Except it wasn't :). Using buildstream 1, you could get either no warning, a warning or an error depending on the version of ruaml-yaml that you're using. This is why I think it is reasonable to merge this even though it used to "work": depending on the version of a dependency it could work or not. -- 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]
