-
Notifications
You must be signed in to change notification settings - Fork 18
PD012 is outdated #120
Copy link
Copy link
Open
Labels
help wantedExtra attention is neededExtra attention is needednew checkNew check for the linterNew check for the linter
Description
First off, thank you for developing this plugin!
Is your feature request related to a problem? Please describe.
read_table is no longer deprecated in favour of read_csv following a discussion. As far as I can tell from reading the references in the given motivation, this means there is no longer a good reason to recommend read_csv over read_table per se.
Describe the solution you'd like
The rule should either be categorised as opinionated to require an opt-in (like PD901), removed, or changed to only emit a warning if read_table is called with sep="," (which is what ruff has just done).
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
help wantedExtra attention is neededExtra attention is needednew checkNew check for the linterNew check for the linter