Fix import formatting using isort
See original GitHub issueFollowing on from #23048. We now need to run isort on all *.py files in the codebase that we are ignoring in CI see here. Information on Isort here
Useful commands:
To sort imports in files in pandas directory ( ignoring those explicitly mentioned in setup.cfg)
isort --recursive pandas
To check imports are formatted correctly:
isort --recursive --check-only pandas
Find a directory you wish to fix from the list below( larger directories by # of files) And comment on here that you are working on fixing. To avoid duplicated efforts.
Directories we need to cover:
- pandas/*.py ( in progress @alimcmaster1 )
- pandas/util/ ( in progress @alimcmaster1 )
- pandas/io/ ( in progress @alimcmaster1 )
- pandas/core/ ( in progress @alimcmaster1 )
- pandas/core/indexes/ ( in progress @alimcmaster1 )
- pandas/core/dtypes/ (@alexander-ponomaroff )
- pandas/core/groupby/ (@alexander-ponomaroff )
- pandas/core/reshape/ (@streeck)
- pandas/core/computation/ (@streeck)
- pandas/tests/indexes/ (@thoo)
- pandas/tests/indexes/multi
- pandas/tests/indexes/interval
- pandas/tests/indexes/timedeltas
- pandas/tests/series/ (@thoo)
- pandas/tests/arrays/ (@thoo)
- pandas/tests/util/ (@thoo )
- pandas/tests/extension/ (@mwoss )
- pandas/tests/io/parser/ (@mwoss )
- pandas/tests/io/formats/
- pandas/tests/io/msgpack/
- pandas/tests/io/*.py
- pandas/tests/scalar/
- pandas/tests/tslibs/
- pandas/tests/groupby/ (@sam-cohan )
- pandas/tests/plotting/ (@pvarshney1729 )
- pandas/tests/frame/ (@pvarshney1729 )
- pandas/tests/indexing/ (@amphy )
PRs must contain no more than 20 files.
For new contributors please first take a look at our Contributing Guide
Pandas-Dev: Could we please add the following labels “Style”, “Good first Issue”, Contributions Welcome" and any more you feel appropriate.
Thanks, Alistair
Issue Analytics
- State:
- Created 5 years ago
- Comments:34 (32 by maintainers)
Top GitHub Comments
@alimcmaster1 Working on
Thanks both @sam-cohan and @streeck for looking to contribute! @streeck put you down initially for
reshape
andcomputation
to give others the chance to contribute like i’ve said above. Thanks,