search

sul-dlss / web-archiving

placeholder for web archiving work
0 stars 0 forks source link

is jobIdLowerBound a good param name? #48

Closed jmartin-sul closed 7 years ago

jmartin-sul commented 7 years ago

see https://github.com/sul-dlss/wasapi-downloader/pull/72#pullrequestreview-40148407

jmartin-sul commented 7 years ago

@nullhandle? we merged sul-dlss/wasapi-downloader#72 so we could unblock ourselves, but are happy to rename the parameter if desired (it's a string constant in one place, so that should be pretty quick/easy).

ndushay commented 7 years ago

@nullhandle is happy with the name. Leave it.