search

stuckyb / gcdl

6 stars 2 forks source link

refactor `fulfillRequestSynchronous()` #57

Open stuckyb opened 2 years ago

stuckyb commented 2 years ago

The implementation of fulfillRequestSynchronous() has ballooned to nearly 250 lines and is basically untestable in its current state. We need to refactor to make it more modular and break it apart into testable units. Comments on some other issues cover elements of this, but I am adding this issue, too, so we can better track progress.

HeatherSavoy-USDA commented 2 years ago

I'm in the middle of moving all date_grain/date stuff out of it