refactor: Removal of request lib from the code

[krynble]

Github issue / Community forum post (link here to close automatically):

[github-actions]

Great PR! Please pay attention to the following items before merging:

Files matching packages/**:

• If fixing bug, added test to cover scenario.
• If addressing forum or Github issue, added link to description.

Files matching packages/**/*.ts:

• Added unit tests to cover new or updated functionality.

Files matching packages/nodes-base/package.json:

• Avoided adding dependencies for nodes if not absolutely necessary.

Make sure to check off this list before asking for review.

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (release-v1@9b9bef6). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files

@@              Coverage Diff              @@
##             release-v1    #6413   +/-   ##
=============================================
  Coverage              ?   28.21%           
=============================================
  Files                 ?     2978           
  Lines                 ?   184958           
  Branches              ?    20260           
=============================================
  Hits                  ?    52193           
  Misses                ?   131983           
  Partials              ?      782           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[ivov]

Can we remove @types/request-promise-native as well?

[krynble]

@ivov unfortunately not yet, the workflow package still relies on it to work and build. The good thing is that this is only necessary for building, not being bundled in the final app

[janober]

Got released with n8n@1.0.5