Fix config file path resolution on windows

ai / size-limit

Calculate the real cost to run your JS app or lib to keep good performance. Show error in pull request if the cost exceeds the limit.

MIT License

6.48k stars 1.82k forks source link

Closed aryaemami59 closed 4 months ago

aryaemami59 commented 4 months ago

[X] Fixes config file path resolution on windows. On windows sometimes you could get Error [ERR_UNSUPPORTED_ESM_URL_SCHEME] which should now be fixed due to usage of pathToFileURL.

aryaemami59 commented 4 months ago

For what it's worth I think I should probably try and make the tests OS agnostic so we can check for potentital issues like this during CI.

ai commented 4 months ago

We can add Windows to CI

aryaemami59 commented 4 months ago

@ai Yeah but the snapshots make the tests fail on windows.