Hi <@U01DYLVUNJE>! I created several PRs in pander...
# contribute
c
Hi @broad-monitor-993! I created several PRs in pandera today (1 is tooling, 1 is AI, 1 is cleaning up the polars api). Could you please take a look when you have time?
👍 2
b
thanks! taking a look
hey @creamy-flower-43589, would you mind linting https://github.com/unionai-oss/pandera/pull/2226?
c
So it actually requires no linting. Note how I mentioned it depends on my tooling PR? That one updates the hooks such that 2226 is now correct
To be more precise - I developed 2226 on top of 2225, and the hooks passed perfectly. Then I submitted it as a separate PR and we got what you see.
Am I missing something?
b
ah okay, I missed those. btw https://github.com/unionai-oss/pandera/pull/2225#pullrequestreview-3929595650 needs to be updated to install
prek
instead of
pre-commit
in the ci
c
Done. Can you please restart CI?
👍 1
b
thanks, merged.
Would you mind rebasing https://github.com/unionai-oss/pandera/pull/2226 real quick? now the lint issues should go away and I believe the unit test failures have been addressed