Skip to content
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

check os compatibility and dependencies_version in prepare_env.sh #99

Merged
merged 10 commits into from
Jan 26, 2021

Conversation

acezen
Copy link
Collaborator

@acezen acezen commented Jan 22, 2021

What do these changes do?

check os compatibility and dependencies_version in prepare_env.sh to improve "Getting Started" experience

Related issue number

Fixes #93

@acezen
Copy link
Collaborator Author

acezen commented Jan 22, 2021

cc/ @yecol @sighingnow @wenyuanyu

Copy link
Collaborator

@sighingnow sighingnow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Not part of this task, but I'm wondering if we suggest users kind rather than minikube on Linux and MacOS as well. Test is needed, though. (not in high pri).

Copy link
Collaborator

@yecol yecol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You may also want to revise the file in /docs, for https://graphscope.io/docs/installation.html

README.md Outdated

For Linux distributions, we provide a script to install the above dependencies and prepare the environment.
For Ubuntu/CentOs/WSL2, we provide a script to install the above dependencies and prepare the environment.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CentOS

@wenyuanyu
Copy link
Collaborator

Maybe also checking the platform's version? like Ubuntu 14, 16, 18?

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
docs/installation.rst Outdated Show resolved Hide resolved
@acezen
Copy link
Collaborator Author

acezen commented Jan 25, 2021

LGTM.

Not part of this task, but I'm wondering if we suggest users kind rather than minikube on Linux and MacOS as well. Test is needed, though. (not in high pri).

thanks for the suggestion. this script can include MacOS prepare environment though kind i think, i will add this feature in next related pr after test.

@sighingnow sighingnow merged commit 6865c1b into alibaba:main Jan 26, 2021
@acezen acezen deleted the zwb/pre_env branch September 3, 2021 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve "Getting Started" experience
4 participants