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.

Sign up for GitHub

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

Jump to bottom

[doc]fix links in build with source #1644

Merged
merged 5 commits into from
Jan 17, 2020
Merged

[doc]fix links in build with source #1644

merged 5 commits into from
Jan 17, 2020

Conversation

Liuxue-Yang
Copy link

as title

darionyaphet
darionyaphet approved these changes Jan 17, 2020
View reviewed changes
Copy link
Contributor

@darionyaphet darionyaphet left a comment

Choose a reason for hiding this comment

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

LGTM

@dutor dutor added the ready-for-testing PR: ready for the CI test label Jan 17, 2020
@dutor dutor merged commit 504f1f5 into vesoft-inc:master Jan 17, 2020
@Liuxue-Yang Liuxue-Yang deleted the fix-link-build-source branch January 17, 2020 03:58
tong-hao pushed a commit to tong-hao/nebula that referenced this pull request Jun 1, 2021
@Liuxue-Yang @dutor
[doc]fix links in build with source (vesoft-inc#1644)
aa71af3 
* fix links in build with source

* update

* update

Co-authored-by: yaphet <darion.wang@vesoft.com>
Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@darionyaphet darionyaphet darionyaphet approved these changes

@amber-moe amber-moe amber-moe approved these changes

@whitewum whitewum Awaiting requested review from whitewum

Assignees
No one assigned
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Liuxue-Yang @darionyaphet @amber-moe @dutor