Skip to content
This repository has been archived by the owner on Feb 6, 2024. It is now read-only.

Commit

Permalink
fix: removes no-param-reassign
Browse files Browse the repository at this point in the history
  • Loading branch information
moltar authored and Roman committed May 18, 2020
1 parent 1767e75 commit 4b840c0
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions src/http.ts
Original file line number Diff line number Diff line change
Expand Up @@ -129,22 +129,20 @@ const canonicalizeParameters = (parameters: CleanParameters): string => {
return sp.toString().replace(/\+/g, '%20')
}

/* eslint-disable no-param-reassign */
const cleanParameters = (parameters: Parameters): CleanParameters =>
Object.entries(parameters)
.filter(([, v]) => v !== undefined)
.reduce((result, [k, v]) => {
if (Array.isArray(v)) {
for (let index = 0; index < v.length; index += 1) {
result[`${k}.${index + 1}`] = String(v)
Object.assign(result, { [`${k}.${index + 1}`]: String(v) })
}
} else {
result[k] = String(v)
Object.assign(result, { [k]: String(v) })
}

return result
}, {} as CleanParameters)
/* eslint-enable no-param-reassign */

const defaultFetch = ({ url, method, headers, data }: Request): Promise<RequestResponse> =>
axios({ method, url, headers, data })
Expand Down

0 comments on commit 4b840c0

Please sign in to comment.