glynnbird commented on code in PR #314: URL: https://github.com/apache/couchdb-nano/pull/314#discussion_r1827552822
########## lib/nano.js: ########## @@ -15,7 +15,7 @@ const assert = require('assert') const stream = require('stream') const Readable = stream.Readable const undici = require('undici') -const fetch = global.fetch || undici.fetch +const fetcher = fetch || global.fetch || undici.fetch Review Comment: I think I was trying (at this point in the PR) trying to get this working on older Node.js's that didn't have fetch built in. It so long to get the PR finished, that fetch was present in all Node.js LTS versions so the code got simplified somewhat -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@couchdb.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org