New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WINDUP-2274 Scripts for local build of CLI and Web UI #3
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happy with changes, scripts run successfully.
Is it worth specifying in the main git title page that you run these scripts from your umbrella 'Projects' dir using eg
./windup-local-build-scripts/build_web.sh
I tried to run them from the windup-local-build-scripts root and of course they failed from there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It worked for me as well, thanks @PhilipCattanach 👍
I would only add, as @m-brophy suggested, the instructions on how to run the scripts in the README.md
file of this project
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@PhilipCattanach I've just added 2 changes to format the commands: if you accept them, then i'll merge this PR.
Co-Authored-By: PhilipCattanach <31246010+PhilipCattanach@users.noreply.github.com>
Co-Authored-By: PhilipCattanach <31246010+PhilipCattanach@users.noreply.github.com>
Run the scripts from the parent folder to your cloned repositories
The build_cli.sh expects the windup, windup-rulesets and windup-distribution projects to have been cloned.
The build_web.sh expects the windup-web, windup-openshift and windup-web-distribution projects to have been cloned.
For the build_web.sh please ensure that you have the docker daemon running