eugene
(Eugene)
June 12, 2020, 3:03pm
1
Does anyone contribute/review PRs for DefinitelyTyped? I could use some eyes on newman types PR.
It looks like newman types were last updated for 4.5. We just added requestAgents in 5.1.0 and I started a PR to add that to types. Anything else that should be included?
DefinitelyTyped:master
← gone-skiing:newman-request-agents
opened 10:50PM - 11 Jun 20 UTC
Please fill in this template.
- [x] Use a meaningful title for the pull reque… st. Include the name of the package modified.
- [x] Test the change in your own code. (Compile and run.)
- [x] Add or edit tests to reflect the change. (Run with `npm test`.)
- [ ] Follow the advice from the [readme](https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/README.md#make-a-pull-request).
- [ ] Avoid [common mistakes](https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/README.md#common-mistakes).
- [x] Run `npm run lint package-name` (or `tsc` if no `tslint.json` is present).
Select one of these and delete the others:
If changing an existing definition:
- [x] Provide a URL to documentation or source code which provides context for the suggested changes: https://github.com/postmanlabs/newman/pull/2392
- [x] If this PR brings the type definitions up to date with a new version of the JS library, update the version number in the header.
- [x] Include [tests for your changes](https://github.com/DefinitelyTyped/DefinitelyTyped#testing)
- [ ] If you are making substantial changes, consider adding a `tslint.json` containing `{ "extends": "dtslint/dt.json" }`. If for reason the any rule need to be disabled, disable it for that line using `// tslint:disable-next-line [ruleName]` and not for whole package so that the need for disabling can be reviewed.
Cheers!