phin issues

Don't parse JSON if status code is not 2xx

Fixes #61 Prevents parsing body as JSON unless the status code is in the 2xx range to allow 4xx and 5xx errors to propagate without JSON parsing errors

jdforsythe

Add eslint to catch errors and enforce style

Fixes these issues: - https://eslint.org/docs/rules/no-prototype-builtins#disallow-use-of-objectprototypes-builtins-directly-no-prototype-builtins - https://eslint.org/docs/rules/no-else-return#disallow-return-before-else-no-else-return - Use `Object.entries()` to get key-value pairs in a loop - Use `Object.assign()` to merge default options (this could be a `{...defaultOpts, ...nops}`...

jdforsythe

Add production dependency audit before publishing

Adds an `auditdeps` npm script to audit production dependencies for vulnerabilities (using the built-in `npm audit --production`) and runs the script before publishing

jdforsythe

Simplify type definitions for overloaded function

1

This just makes the type definitions simpler for the overloaded `phin()` function

jdforsythe

Documentation

Wrong type on response body

4

Hi 👋 I think there's a problem with either the response body parsing or the type definitions. ### Expected Looking at type definitions, the response body is of type `string`...

fthouraud

Documentation

Contribution Guide

1

Hello, I was going to open a PR for both phin and centra and add a few improvements: * Replace `forEach` with `for-loop` for performance reasons * Remove unnecessary `else`...

ktalebian

Feature

Add slim-fetch to comparison

Slim-fetch is a minified distribution of node-fetch. It greatly reduces the package size associated with fetch while also including everything in the [WHATWG fetch standard](https://fetch.spec.whatwg.org/). Because it is the in...

leonm1