-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
docs(python): Update reference to apply
#15982
Conversation
Removed the reference to `apply` as indicated in #15945 and replaced with `map_rows` or `map_elements` as appropriate.
Test failures unrelated to this PR: Ruff job failed to setup and Code Coverage failed with the error:
|
Nice one; you receive today's "Eye of the Tiger" award 😁🏆 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh; just to be sure I re-ran the failing ruff
lint job and one small fix is required ;)
py-polars/polars/dataframe/frame.py:6202:89:
W505 Doc line too long (89 > 88)
Haha, full credit goes to @david-waterworth though :)
Nice catch, have made the update. Code coverage seems to require a token now, so I guess that will fail until then. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #15982 +/- ##
=======================================
Coverage 80.91% 80.91%
=======================================
Files 1384 1384
Lines 178111 178111
Branches 3043 3043
=======================================
+ Hits 144112 144120 +8
+ Misses 33515 33506 -9
- Partials 484 485 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Thank you both! ✌️ |
Closes #15945.
Removed the reference to
apply
as indicated and replaced withmap_rows
ormap_elements
as appropriate.