Add @types/multer as peerDependency

[azven-aumni]

trafficstars

Sorting

• I'm submitting a ...

  • [x] bug report
  • [ ] feature request
  • [ ] support request

• I confirm that I

  • [x] used the search to make sure that a similar issue hasn't already been submit

Expected Behavior

The yarn build CircleCI test should pass.

Current Behavior

Possible Solution

I'm not aware of any solutions, so either a bug fix for this issue or a workaround would be appreciated.

Steps to Reproduce

1. A Snyk scan has identified that [email protected] had vulnerable dependencies.
2. I created a PR to upgrade to [email protected], which is then ran through a number of CircleCI tests.
3. The last test yarn build fails due to the following error: node_modules/@tsoa/runtime/dist/config.d.ts:43:18 - error TS2709: Cannot use namespace 'MulterOpts' as a type.
4. The PR cannot be approved and merged.

Context (Environment)

Version of the library: 4.0.0 Version of NodeJS: 16.14.2

• Confirm you were using yarn not npm: [x]

Detailed Description

A fix for error TS2709.

Breaking change?

No

[github-actions[bot]]

Hello there azven-aumni 👋

Thank you for opening your very first issue in this project.

We will try to get back to you as soon as we can.👀

[WoH]

This happens if you don't have @types/multer in your dependencies. I think we could require these types at least as a peerDep.

[RobbeCl]

@WoH Just a question: why wouldn't we want to import it as a dependency since we are importing it in packages/runtime/src/config.ts

[dxphilo]

Would love to pick up this issue if it is not yet assigned to anybody. @WoH

[WoH]

Please go ahead