Merge pull request #2776 from pre-commit/unsorted-additional-deps

remove sorting for repo key for additional_deps
pre-commit · Feb 23, 2023 · 7bfa7e1 · 7bfa7e1
2 parents cdd3606 + a631abd
commit 7bfa7e1
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/pre_commit/repository.py b/pre_commit/repository.py
@@ -33,7 +33,7 @@ def _state_filename_v2(venv: str) -> str:
 
 
 def _state(additional_deps: Sequence[str]) -> object:
-    return {'additional_dependencies': sorted(additional_deps)}
+    return {'additional_dependencies': additional_deps}
 
 
 def _read_state(venv: str) -> object | None:

diff --git a/pre_commit/store.py b/pre_commit/store.py
@@ -125,7 +125,7 @@ def connect(
     @classmethod
     def db_repo_name(cls, repo: str, deps: Sequence[str]) -> str:
         if deps:
-            return f'{repo}:{",".join(sorted(deps))}'
+            return f'{repo}:{",".join(deps)}'
         else:
             return repo
 

diff --git a/tests/store_test.py b/tests/store_test.py
@@ -180,7 +180,7 @@ def test_create_when_store_already_exists(store):
 
 def test_db_repo_name(store):
     assert store.db_repo_name('repo', ()) == 'repo'
-    assert store.db_repo_name('repo', ('b', 'a', 'c')) == 'repo:a,b,c'
+    assert store.db_repo_name('repo', ('b', 'a', 'c')) == 'repo:b,a,c'
 
 
 def test_local_resources_reflects_reality():