Skip to content

fix: typo for the builder error type - #24052

Open
JosephLenton wants to merge 2 commits into
apache:mainfrom
JosephLenton:fix-plan-builder-data-type-error-typo
Open

fix: typo for the builder error type#24052
JosephLenton wants to merge 2 commits into
apache:mainfrom
JosephLenton:fix-plan-builder-data-type-error-typo

Conversation

@JosephLenton

@JosephLenton JosephLenton commented Aug 2, 2026

Copy link
Copy Markdown

Which issue does this PR close?

There is no ticket for this (I can open one if it is really needed). This is a very small nitpick QoL improvement, by changing the wording for an error.

The typo fixes the phrase ... can't cast to got ..., by adding a comma and making the rest a bit clearer.

Rationale for this change

  • The error doesn't read as natural english with ... can't cast to got ....
  • I've added more clarity on what got and for are, by changing to data of type and field of type.

What changes are included in this PR?

  • Rewording an error message.

Are these changes tested?

  • I added a test to confirm the error message looks right.

Are there any user-facing changes?

  • A user facing error message has changed.
  • No documentation changes are needed.
  • No API breaking changes are in this PR.

@github-actions github-actions Bot added the logical-expr Logical plan and expressions label Aug 2, 2026
@JosephLenton
JosephLenton force-pushed the fix-plan-builder-data-type-error-typo branch from 24b04be to 5a0c76f Compare August 2, 2026 11:01
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.85%. Comparing base (541caab) to head (5a0c76f).
⚠️ Report is 25 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #24052      +/-   ##
==========================================
+ Coverage   80.84%   80.85%   +0.01%     
==========================================
  Files        1096     1101       +5     
  Lines      373936   375446    +1510     
  Branches   373936   375446    +1510     
==========================================
+ Hits       302313   303582    +1269     
- Misses      53584    53768     +184     
- Partials    18039    18096      +57     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@JosephLenton
JosephLenton force-pushed the fix-plan-builder-data-type-error-typo branch from 445d9cb to bd7234c Compare August 3, 2026 15:53
@JosephLenton
JosephLenton marked this pull request as ready for review August 3, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

logical-expr Logical plan and expressions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants