infra(netlify): remove ignore script

[ST-DDT]

Removes the netlify ignore check.

• There are some cases where it would be nice if there was a preview of the changes (aka as some dependency changes).
• The ignore script does not reduce the number of notifications and we aren't close to the build limits either

[netlify]

✅ Deploy Preview for fakerjs ready!

Name	Link
🔨 Latest commit	fe138ad
🔍 Latest deploy log	https://app.netlify.com/sites/fakerjs/deploys/66df6c15da42b50009078091
😎 Deploy Preview	https://deploy-preview-3092.fakerjs.dev
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.96%. Comparing base (533c131) to head (fe138ad).
Report is 1 commits behind head on next.

Additional details and impacted files

@@           Coverage Diff            @@
##             next    #3092    +/-   ##
========================================
  Coverage   99.96%   99.96%            
========================================
  Files        2776     2776            
  Lines      226260   226260            
  Branches      590      942   +352     
========================================
+ Hits       226190   226191     +1     
+ Misses         70       69     -1     

see 1 file with indirect coverage changes

[Shinigami92]

There are some cases where it would be nice if there was a preview of the changes (aka as some dependency changes).

So should we just remove pnpm-lock.yaml from the check?

[ST-DDT]

There are some cases where it would be nice if there was a preview of the changes (aka as some dependency changes).

So should we just remove pnpm-lock.yaml from the check?

Then it triggers in 75% of cases, so there is no need for the added complexity for no gain.
(The difference is only the locale PRs)