Remove dependency on isomorphic-fetch

[Yushmanth-reddy]

Which problem is this PR solving?

• Helps with [deps] Help with upgrading eslint-plugin-import #1613

Short description of the changes

• remove dependency on isomorphic-fetch

[codecov]

Codecov Report

Patch and project coverage have no change.

Comparison is base (f559122) 96.01% compared to head (3d79af3) 96.01%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1616   +/-   ##
=======================================
  Coverage   96.01%   96.01%           
=======================================
  Files         242      242           
  Lines        7559     7559           
  Branches     1984     1984           
=======================================
  Hits         7258     7258           
  Misses        301      301           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[anshgoyalevil]

Was that needed to be moved from devDependencies to dependencies?

[Yushmanth-reddy]

after changing scripts/get-changelog.js there was an error ('isomorphic-fetch' should be listed in the project's dependencies, not devDependencies). So I modified it and it worked fine.

[yurishkuro]

Can we remove this dep altogether? It's a polyfill for browsers, but we're using this script only with node.js

[anshgoyalevil]

@yurishkuro Yes, it can be removed. @Yushmanth-reddy Here's some instructions to go ahead

// ... (rest of the code)

const { readFile } = require('fs').promises; // Use native fs.promises.readFile

// ... (rest of the code)

// Remove the following line since 'promisify' is not required anymore
// const { promisify } = require('util');

// ... (rest of the code)

// Modify this function to use native fs.promises.readFile
// const readFilePromise = promisify(readFile);
// Instead, use the following line to define readFilePromise
// const readFilePromise = readFile;

// ... (rest of the code)

// No other changes are required in the code.

[Yushmanth-reddy]

Sure, thanks @anshgoyalevil

[yurishkuro]

It's still not removed

[Yushmanth-reddy]

Don't know why the unit test is failed but the command "yarn install --frozen-lockfile" is running successfully in my local setup.

[Yushmanth-reddy]

@yurishkuro Should I make any changes?