Skip to content

Commit 4dbf371

Browse files
authored
Merge pull request #22 from kurtmckee/expand-to-100pct-coverage
Remove an unnecessary conditional
2 parents 9b5469e + ebdeb5f commit 4dbf371

File tree

2 files changed

+6
-7
lines changed

2 files changed

+6
-7
lines changed

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ source = [
4949

5050
[tool.coverage.report]
5151
skip_covered = true
52-
fail_under = 99
52+
fail_under = 100
5353

5454
[tool.coverage.html]
5555
directory = "htmlcov/"

sqids/sqids.py

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -110,13 +110,12 @@ def decode(self, id_: str) -> List[int]:
110110
while id_:
111111
separator = alphabet[0]
112112
chunks = id_.split(separator)
113-
if chunks:
114-
if not chunks[0]:
115-
return ret
113+
if not chunks[0]:
114+
return ret
116115

117-
ret.append(self.__to_number(chunks[0], alphabet[1:]))
118-
if len(chunks) > 1:
119-
alphabet = self.__shuffle(alphabet)
116+
ret.append(self.__to_number(chunks[0], alphabet[1:]))
117+
if len(chunks) > 1:
118+
alphabet = self.__shuffle(alphabet)
120119

121120
id_ = separator.join(chunks[1:])
122121

0 commit comments

Comments
 (0)