Closed sebmartin closed 5 years ago
Lines need to be a bit shorter, otherwise LGTM
@kmtaylor-github I bumped the batch size to 5000 as discussed. Is this what you had in mind? 50k seemed a little extreme and could add memory pressure.
Remaining/existing CI failures are handled in a separate PR.
Surface the
details
flag to avoid making additional REST calls (1 per item in the list) when the extra details are not necessary.