Skip to content

Minor: Change all file links to GitHub in document#7768

Merged
jackwener merged 1 commit into
apache:mainfrom
ongchi:doc-update-file-links
Oct 8, 2023
Merged

Minor: Change all file links to GitHub in document#7768
jackwener merged 1 commit into
apache:mainfrom
ongchi:doc-update-file-links

Conversation

@ongchi

@ongchi ongchi commented Oct 7, 2023

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes #.

Rationale for this change

Folder links does not work in the document.

e.g
https://arrow.apache.org/datafusion/user-guide/example-usage.html#example-usage

more code examples link in Example Usage section does not works at all.

What changes are included in this PR?

Change all file links to GitHub in document, which would be easier to browse.

Are these changes tested?

Are there any user-facing changes?

@alamb alamb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you very mcuch @ongchi 🙏

@jackwener jackwener left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I have verified that all ../../../datafusion-example is corrected.
Thanks @ongchi .

@jackwener jackwener merged commit 5e9dd44 into apache:main Oct 8, 2023
@ongchi ongchi deleted the doc-update-file-links branch October 8, 2023 09:05
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.

3 participants