FatalLin commented on PR #1029: URL: https://github.com/apache/submarine/pull/1029#issuecomment-1342858350
> > > @FatalLin I think this should be fine, but I just have a question: Can we add a version range limit on the changes like @pingsutw mentioned last time? This would avoid the problem of new versions causing build failures again in the future. > > > > > > do you mean fix the version of all dependency? > > I mean the lines you changed with `pip install`. I think we could fix the version at 1.2.2 - which would be the same with the current command, Im not favor in using a range of version on single dependency is because somehow it may bring unexpected changes from their own dependency, just like it require the version of nodeJs is higher than 14.0 this time. -- 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]
