EasyPost / easypost-node

85%

push

github

feat: typescript testing, correcting typescript errors (#542)

# Description

- Adds missing Typescript params for FedEx MFA
- I realized that we don't actually test if the Typescript definitions
in this project are valid, we now run the Typescript compiler (`just
typescript`) to ensure syntax errors can't pass into prod (doing so
exposed a couple small issues that needed to be fixed in the
definitions)
- Bumps dependencies to get Typescript happy with config added
- Removes a couple of lingering references to the removed `repl`
- Bumps version to release FedEx MFA

<!-- Please provide a general summary of your PR changes and link any
related issues or other pull requests. -->

# Testing

`just typescript` now runs on CI to catch future errors, fixed the ones
it caught here.
<!--
Please provide details on how you tested this code. See below.

- All pull requests must be tested (unit tests where possible with
accompanying cassettes, or provide a screenshot of end-to-end testing
when unit tests are not possible)
- New features must get a new unit test
- Bug fixes/refactors must re-record existing cassettes
-->

# Pull Request Type

Please select the option(s) that are relevant to this PR.

- [ ] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [ ] Improvement (fixing a typo, updating readme, renaming a variable
name, etc)

187 of 275 branches covered (68.0%)

Branch coverage included in aggregate %.

782 of 862 relevant lines covered (90.72%)

245.28 hits per line