Skip to content

docs: correct the comment in util.py#5252

Merged
yanghua merged 1 commit intolance-format:mainfrom
shiyajuan123:feat-optimize-comment
Nov 17, 2025
Merged

docs: correct the comment in util.py#5252
yanghua merged 1 commit intolance-format:mainfrom
shiyajuan123:feat-optimize-comment

Conversation

@shiyajuan123
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added enhancement New feature or request python labels Nov 17, 2025
@yanghua yanghua changed the title feat: optimize the comments in util.py doc: correct the comment in util.py Nov 17, 2025
@github-actions
Copy link
Copy Markdown
Contributor

ACTION NEEDED
Lance follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message. Please update your PR title and description to match the specification.

For details on the error please inspect the "PR Title Check" action.

@yanghua
Copy link
Copy Markdown
Collaborator

yanghua commented Nov 17, 2025

Hi @shiyajuan123, thanks for your contribution. Please align the commit message with the title of the PR.

Copy link
Copy Markdown
Collaborator

@yanghua yanghua left a comment

Choose a reason for hiding this comment

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

LGTM, will merge after CI and commit msg.

@shiyajuan123 shiyajuan123 changed the title doc: correct the comment in util.py docs: correct the comment in util.py Nov 17, 2025
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Nov 17, 2025
@yanghua yanghua merged commit 975c59c into lance-format:main Nov 17, 2025
13 of 14 checks passed
jackye1995 pushed a commit to jackye1995/lance that referenced this pull request Jan 21, 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 enhancement New feature or request python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants