custom-elements-manifest icon indicating copy to clipboard operation
custom-elements-manifest copied to clipboard

fix: path parsing on Windows (non-unix)

Open bheston opened this issue 2 years ago • 1 comments

Fix an issue running analyzer in a monorepo with a reference to custom elements in another package. The find-dependencies package converts the path to Unix format, but then uses path.sep to parse the path. On Windows the separators are not the same so it fails.

I tried to add a test for this case, but that branch needs to read the package.json file which I don't believe can happen in the mocked test environment.

bheston avatar Feb 19 '24 00:02 bheston

Deploy Preview for custom-elements-manifest-analyzer ready!

Name Link
Latest commit 3683e7a9a78a02e7035832e0f2c7980bf5cd5ebc
Latest deploy log https://app.netlify.com/sites/custom-elements-manifest-analyzer/deploys/65d2a172a02f0d00080e6d50
Deploy Preview https://deploy-preview-235--custom-elements-manifest-analyzer.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

netlify[bot] avatar Feb 19 '24 00:02 netlify[bot]