Skip to content
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.

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

feat: Improve error message on SchemaError #20326

Merged
merged 1 commit into from
Dec 17, 2024
Merged

Conversation

kszlim
Copy link
Contributor

@kszlim kszlim commented Dec 17, 2024

I know you want to do better typechecking and error messages at the IR level, but meanwhile this is a monotonic and small change to improve the current state of things.

@kszlim
Copy link
Contributor Author

kszlim commented Dec 17, 2024

Partially fixes:
#20322

@kszlim kszlim changed the title Improve error message on SchemaError fix: Improve error message on SchemaError Dec 17, 2024
@github-actions github-actions bot added fix Bug fix python Related to Python Polars rust Related to Rust Polars and removed title needs formatting labels Dec 17, 2024
@kszlim kszlim changed the title fix: Improve error message on SchemaError fix(rust): Improve error message on SchemaError Dec 17, 2024
Copy link

codecov bot commented Dec 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.61%. Comparing base (2fc26cf) to head (7a4f447).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #20326   +/-   ##
=======================================
  Coverage   79.60%   79.61%           
=======================================
  Files        1567     1567           
  Lines      218528   218528           
  Branches     2462     2462           
=======================================
+ Hits       173969   173975    +6     
+ Misses      43992    43986    -6     
  Partials      567      567           

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

@kszlim
Copy link
Contributor Author

kszlim commented Dec 17, 2024

I believe the title label is correct now?

@ritchie46 ritchie46 changed the title fix(rust): Improve error message on SchemaError feat: Improve error message on SchemaError Dec 17, 2024
@ritchie46 ritchie46 merged commit 80f8945 into pola-rs:main Dec 17, 2024
23 of 24 checks passed
@github-actions github-actions bot added the enhancement New feature or an improvement of an existing feature label Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or an improvement of an existing feature fix Bug fix python Related to Python Polars rust Related to Rust Polars
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants