earthkit-data icon indicating copy to clipboard operation
earthkit-data copied to clipboard

fix: Add an explicit length check for `did_you_mean`

Open HCookie opened this issue 2 weeks ago • 1 comments

Description

Add an explicit length check for did_you_mean

Todo

  • [ ] add tests

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

HCookie avatar Nov 20 '25 17:11 HCookie

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 85.65%. Comparing base (ce31e67) to head (50430b1). :warning: Report is 4 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #855   +/-   ##
========================================
  Coverage    85.65%   85.65%           
========================================
  Files          182      182           
  Lines        14287    14287           
  Branches       709      709           
========================================
  Hits         12238    12238           
  Misses        1844     1844           
  Partials       205      205           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov-commenter avatar Nov 20 '25 18:11 codecov-commenter