Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch from naturalsort to natsort #1318

Merged
merged 1 commit into from
Jan 22, 2022
Merged

Switch from naturalsort to natsort #1318

merged 1 commit into from
Jan 22, 2022

Conversation

dvzrv
Copy link
Contributor

@dvzrv dvzrv commented Jan 21, 2022

pyocd/commands/commands.py:
Use natsort's natsorted() instead of naturalsort's natsort().

setup.cfg:
Replace naturalsort with natsort.

Fixes #1317

Copy link
Member

@flit flit left a comment

Choose a reason for hiding this comment

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

Since it's only a 2 line change in the .py file, I won't make it a strict requirement that you add your copyright to the file header, but it would be nice.

@pyocd pyocd deleted a comment from azure-pipelines bot Jan 21, 2022
@dvzrv
Copy link
Contributor Author

dvzrv commented Jan 21, 2022

Sure, I can add it, no problem.

I hope my change works as intended!

pyocd/commands/commands.py:
Use natsort's `natsorted()` instead of naturalsort's `natsort()`.

setup.cfg:
Replace naturalsort with natsort.

Fixes pyocd#1317
@flit
Copy link
Member

flit commented Jan 21, 2022

Thanks!

I just did a quick test, and it works as expected. 👍🏽

@flit flit merged commit ef01536 into pyocd:main Jan 22, 2022
flit pushed a commit to flit/pyOCD that referenced this pull request Jan 22, 2022
pyocd/commands/commands.py:
Use natsort's `natsorted()` instead of naturalsort's `natsort()`.

setup.cfg:
Replace naturalsort with natsort.

Fixes pyocd#1317
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace naturalsort dependency with natsort or remove altogether
2 participants