-
Notifications
You must be signed in to change notification settings - Fork 1
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
Revise build.sh, add initial README.md #1
Conversation
f7c5da6
to
4604921
Compare
build.sh
Outdated
else | ||
fault | ||
fi | ||
if [ "$1" = "linux" ]; then |
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.
Please do not modify the main function, this will destroy github action
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.
Very sorry about that.
How do you think about also updating the .github/workflow/build.yml
like below?
- bash ./build.sh build all
+ bash ./build.sh all
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.
Would like to keep the command categorized
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.
I see, will update the patch to stick to original usage.
d072c37
to
64fd7b7
Compare
Signed-off-by: Yanfeng Liu <[email protected]>
Signed-off-by: Yanfeng Liu <[email protected]>
This PR revises help message of build.sh so that to sync with the code.
It also adds an initial version of README file to help people get started quickly.