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

Revise document of installing ballista pinned to specified version #2034

Merged
merged 1 commit into from
Mar 19, 2022

Conversation

WinkerDu
Copy link
Contributor

@WinkerDu WinkerDu commented Mar 18, 2022

Which issue does this PR close?

Closes #1809.

Rationale for this change

To make sure the ballista install examples in docs/source/user-guide/distributed/deployment/cargo-install.md and ballista/rust/client/README.md always work irrespective of crate updates

What changes are included in this PR?

No.

Are there any user-facing changes?

No.

@WinkerDu
Copy link
Contributor Author

cc @xudong963 , can you please approve running workflows?

@xudong963 xudong963 requested a review from houqp March 19, 2022 03:13
@xudong963 xudong963 added the documentation Improvements or additions to documentation label Mar 19, 2022
@WinkerDu
Copy link
Contributor Author

@houqp Can you please review this patch? thank you

@houqp houqp merged commit 12996ce into apache:master Mar 19, 2022
@WinkerDu WinkerDu deleted the revise-ballista-example-doc2 branch March 21, 2022 03:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot install ballista-scheduler
3 participants