Skip to content

docs: fix issues in HF integration docs#5778

Merged
jackye1995 merged 1 commit intomainfrom
feat/refactor-hf-docs
Jan 22, 2026
Merged

docs: fix issues in HF integration docs#5778
jackye1995 merged 1 commit intomainfrom
feat/refactor-hf-docs

Conversation

@prrao87
Copy link
Copy Markdown
Contributor

@prrao87 prrao87 commented Jan 22, 2026

  • This PR addresses the issues in sidebar organization for the Hugging Face integration. Now, the HF docs should appear in two pages and are more clearly visible in the page at first glance.
  • Remove redundant old sidebar link for huggingface.md (the doc source files come straight from the lance-format/lance-huggingface repo)

@prrao87 prrao87 requested a review from jackye1995 January 22, 2026 03:19
@prrao87 prrao87 changed the title Fix issues in HF integration docs docs: fix issues in HF integration docs Jan 22, 2026
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Jan 22, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Code Review

This PR looks good. The changes correctly:

  1. Move the Huggingface docs copy step earlier in the workflow (before Spark/Ray)
  2. Remove the stale static reference to huggingface.md from .pages since the content is dynamically copied from the lance-huggingface repo

No issues found. The sidebar ordering change should now reflect the intended documentation structure.

@lance-format lance-format deleted a comment from github-actions Bot Jan 22, 2026
@jackye1995 jackye1995 merged commit 54744d9 into main Jan 22, 2026
7 of 8 checks passed
@jackye1995 jackye1995 deleted the feat/refactor-hf-docs branch January 22, 2026 03:21
jackye1995 pushed a commit to jackye1995/lance that referenced this pull request Jan 22, 2026
wojiaodoubao added a commit to wojiaodoubao/lance that referenced this pull request Jan 22, 2026
majin1102 pushed a commit to majin1102/lance that referenced this pull request Jan 23, 2026
vivek-bharathan pushed a commit to vivek-bharathan/lance that referenced this pull request Feb 2, 2026
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.

2 participants