syermakov commented on pull request #18: URL: https://github.com/apache/incubator-nlpcraft/pull/18#issuecomment-901314341
@rahul3 thank you very much for sharing your excellent work! @skhdl thank you for detailed review! Let me also join the discussion of Ctxserver installation. I think we should keep in mind two use cases: 1. Development process 2. End user experience For us, as developers, it is easier to run Maven commands in order to build and verify changes. But for end users it can be a tedious task to setup all the necessary tools to build the whole project. That is one of the reasons of why we have binary releases that users can use to quickly download and run the NLPCarft server out of the box. In my opinion we should have installation scripts that would help end users to install all the necessary dependencies. We can also run those scripts via Maven to make developers life easier and to make sure that those scripts get executed as part of our build/verify process. -- 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]
