Closed swarkentin closed 2 years ago
It looks like run IDs have exceeded the i32 max.
i32
https://github.com/softprops/hubcaps/blob/a9e6616ecc7b8eef0ea0eaf6055d9610c8961500/src/checks.rs#L186
This is a recent example (run id = 2280865524):
2280865524
https://github.com/ForAllSecure/api-fuzzer-swagger-petstore-demo/runs/2280865524
This fix will break consumers of CheckRun, but bumping to a u64 or replacing the id with a String is necessary to work with any new runs.
CheckRun
u64
String
It looks like run IDs have exceeded the
i32max.https://github.com/softprops/hubcaps/blob/a9e6616ecc7b8eef0ea0eaf6055d9610c8961500/src/checks.rs#L186
This is a recent example (run id =
2280865524):https://github.com/ForAllSecure/api-fuzzer-swagger-petstore-demo/runs/2280865524
This fix will break consumers of
CheckRun, but bumping to au64or replacing the id with aStringis necessary to work with any new runs.