Skip to content

Commit

Permalink
Using force_alphabetical_sort_within_sections to disable case sensiti…
Browse files Browse the repository at this point in the history
…ve sort order within sections

Summary:
Meta:
Why are we making this change?
To match the convention of other codebases in Meta.
Reference:
https://fb.workplace.com/groups/pyfmt/posts/925830131387837/
D35553814 (facebookresearch@7d6a682)

Why are we not moving to usort yet?
1. We have other customizations in ./dev/.isort.cfg like
2. usort does not seem to be installed by default on the on-demand instances.

Reviewed By: mannatsingh, sdebnathusc

Differential Revision: D35629371

fbshipit-source-id: 6aae7bc9fd20f139ff5bb5ed9a5f30a961b77cb9
  • Loading branch information
Vaibhav Aggarwal authored and facebook-github-bot committed Apr 14, 2022
1 parent be7ca86 commit 6c83054
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions dev/.isort.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,6 @@ default_section = THIRDPARTY
lines_after_imports = 2
combine_as_imports = True
skip=pycls/datasets/data
# Using force_alphabetical_sort_within_sections to match other Meta codebase
# convention.
force_alphabetical_sort_within_sections = True

0 comments on commit 6c83054

Please sign in to comment.