Skip to content

Adds 88. Merge Sorted Array's solution and further improvements - #97

Merged
mathusanm6 merged 2 commits into
mainfrom
fix-lint
Oct 2, 2026
Merged

mathusanm6 merged 2 commits into
mainfrom
fix-lint

Conversation

@mathusanm6

Copy link
Copy Markdown
Owner

This pull request introduces the solution for the "Merge Sorted Array" problem in both Python and C++, adds corresponding test cases, and updates repository metadata and statistics. It also standardizes type annotations in several Python files to use built-in generics, and includes minor cleanup in test files.

Addition of "Merge Sorted Array" problem:

  • Added new directory problems/merge_sorted_array with the following:
    • Python solution in merge_sorted_array.py implementing in-place merging.
    • C++ solution in merge_sorted_array.cc and header in merge_sorted_array.h. [1] [2]
    • Problem metadata in config.yml.
    • Test cases for both Python and C++ solutions. [1] [2]

Repository metadata and documentation updates:

  • Updated .python-version to Python 3.14.8.
  • Updated solution counts and added the new problem to the main table in README.md. [1] [2]

Standardization and cleanup of Python type annotations:

  • Replaced typing.List and similar imports with built-in generics (e.g., list[int], list[str]) in problem solution files:
    • brace_expansion.py, calculate_amount_paid_in_taxes.py, contains_duplicate.py, evaluate_division.py, group_anagrams.py, minimum_flips_in_binary_tree_to_get_result.py. [1] [2] [3] [4] [5] [6]

Minor test and import cleanups:

  • Removed unused imports and adjusted import order in several test files for consistency:
    • brace_expansion_test.py, calculate_amount_paid_in_taxes_test.py, contains_duplicate_test.py, evaluate_division_test.py, group_anagrams_test.py, minimum_flips_in_binary_tree_to_get_result_test.py. [1] [2] [3] [4] [5] [6]

@mathusanm6
mathusanm6 requested a balanced review from Copilot October 2, 2026 10:23
@mathusanm6 mathusanm6 self-assigned this Oct 2, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Fix the failing test import path setup and handle malformed YAML shapes in README generation.

Review effort: Lite
Findings: 1 High severity · 1 Medium severity

Open (2)

Comment thread scripts/generate_readme.py
@mathusanm6
mathusanm6 merged commit 320dd91 into main Oct 2, 2026
3 checks passed
@mathusanm6
mathusanm6 deleted the fix-lint branch October 2, 2026 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants