Skip to content

Deprecate parameter skiprows to skip_rows (Will be removed in v0.20.0) - #4295

Closed
yvonnefroehlich wants to merge 1 commit into
mainfrom
deprecate-skiprows
Closed

Deprecate parameter skiprows to skip_rows (Will be removed in v0.20.0)#4295
yvonnefroehlich wants to merge 1 commit into
mainfrom
deprecate-skiprows

Conversation

@yvonnefroehlich

@yvonnefroehlichyvonnefroehlich commented Dec 16, 2025

Copy link
Copy Markdown
Member

@yvonnefroehlichyvonnefroehlich added this to the 0.18.0 milestone Dec 16, 2025
@yvonnefroehlichyvonnefroehlich self-assigned this Dec 16, 2025
@yvonnefroehlichyvonnefroehlich added the deprecation Deprecating a feature label Dec 16, 2025
@yvonnefroehlichyvonnefroehlich changed the title Deprecate parameter skiprows to skip_rowsDeprecate parameter skiprows to skip_rows (Will be removed in v0.20.0)Dec 16, 2025
@seisman

Copy link
Copy Markdown
Member

This is a common parameter. Do we really want to deprecate it?

@yvonnefroehlich

Copy link
Copy Markdown
MemberAuthor

This is a common parameter. Do we really want to deprecate it?

Hm, skiprows is used in five functions, which is not optimal. "skip" and "rows" are short words, so readability should be OK without an underscore. So, probably better to keep it as is.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deprecationDeprecating a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@yvonnefroehlich@seisman