Skip to content

Improve: Handle empty sequence case in linear_search #14627

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.

Sign up for GitHub

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

Open
snehapriy958 wants to merge 5 commits into
base: master
Choose a base branch
from
+13 −16
Open

Improve: Handle empty sequence case in linear_search #14627

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
1c57978
Fix: improve docstrings and use ValueError in rec_linear_search
snehapriy958 May 4, 2026
234f50b
Improve: handle empty sequence case in linear_search
snehapriy958 May 4, 2026
2ffa363
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] May 4, 2026
28fff0f
Fix: Improve: handle empty sequence case in linear_search
snehapriy958 May 4, 2026
a3425cf
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] May 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[pre-commit.ci] auto fixes from pre-commit.com hooks 
for more information, see https://pre-commit.ci
  • Loading branch information
@pre-commit-ci
pre-commit-ci[bot] committed May 4, 2026
commit 2ffa363b5e4037b39e48ba2e9dea98bd34238d96
2 changes: 2 additions & 0 deletions searches/linear_search.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,14 +7,16 @@
For manual testing run:
python3 linear_search.py
"""


def linear_search(sequence: list, target: int) -> int:

Check failure on line 12 in searches/linear_search.py

View workflow job for this annotation

GitHub Actions / ruff

ruff (ARG001) 

searches/linear_search.py:12:35: ARG001 Unused function argument: `target`
"""... existing docstring ..."""

if not sequence:
return -1


def linear_search(sequence: list, target: int) -> int:

Check failure on line 19 in searches/linear_search.py

View workflow job for this annotation

GitHub Actions / ruff

ruff (F811) 

searches/linear_search.py:19:5: F811 Redefinition of unused `linear_search` from line 12: `linear_search` redefined here
  searches/linear_search.py:12:5: previous definition of `linear_search` here
  help: Remove definition: `linear_search`
"""A pure Python implementation of a linear search algorithm

:param sequence: a collection with comparable items (No sorting required)
Expand Down
Loading