Closed jdforsythe closed 7 months ago
:warning: We detected 8 security issues in this pull request:
👉 Go to the dashboard for detailed results.
📥 Happy? Share your feedback with us.
Hi @jdforsythe Yarn.lock or package-lock both files can't be considered in a PR due to security reasons so they need to be removed from the PR.
@amitguptagwl this has been fixed
Thanks for this change. I'll confirm and publish soon.
Purpose / Goal
See #612
This changes the type definitions to set all options with defaults to optional in the typings.
It adds documentation to all types so they show up in Intellisense when using the types.
It adds a type test - defining objects as the various options types (parse, validate, build) with none of the optional properties. Simply run
npm test-types
to attempt to compile the TS file and it will emit any type errors.Example with no errors:
Example with an error - add
someNewProperty
to the type definition as non-optional:fxp.d.ts:
The type test will now fail since the
typings-test.ts
file does not include this property in the validation options and it's listed as required:This will ensure the test fails if a property is added to the type as non-optional but not added to the
typings-test.ts
file as a required property.Type
Please mention the type of PR
Note : Please ensure that you've read contribution guidelines before raising this PR. If your PR is in progress, please prepend
[WIP]
in PR title. Your PR will be reviewed when[WIP]
will be removed from the PR title.Bookmark this repository for further updates.