pagure

#4658 Support arrow >= 0.15.0
Merged by pingou. Opened by jlanda.
jlanda/pagure support-arrow-015  into  master

Download 4658.patch

Fixes #4611

This should fix our tests, but I wanna check that we are capturing all the potential ValueError throws from arrow before merging this

rebased onto 3ef2feb5af6192c1c15ca256049a013075da71c4

1 new commit added

  • pagure/api/user: capture ValueError exception on arrow.get()

Okido, all the arrow.get that uses user provided strings as argument have a try catch block around them.

I did not add them on git|libgit2 provided times.
Edited by jlanda

Let's rebase this but this is a way simpler diff than I expected, it's nice :)

:thumbsup:

Let's rebase this but this is a way simpler diff than I expected, it's nice :)

Yeah, I expected much more work too, but pagure-admin already has general catches around arrow.get()s and there are no so much of them on api|ui :)
Edited by jlanda

rebased onto b1cfc82fefbe42d731850bea5661bb6eafedbbc8

Let's get this in! :)

Thanks for the patch!!

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.