Replace path with pathe - Githubissues

cursorless-dev / cursorless

Don't let the cursor slow you down

https://www.cursorless.org/

MIT License

1.13k stars 78 forks source link

Replace path with pathe #2481

Closed AndreasArvidsson closed 2 months ago

AndreasArvidsson commented 2 months ago

This is a simple way of getting rid of node path references in common and engine. I thought it best to also replace it in vscode so we don't have different implementations for path in the bundle. I guess for things like running tests and ours setup scrips then just using node path is probably fine? I could go either way.

https://www.npmjs.com/package/pathe

Checklist

[/] I have added tests
[/] I have updated the docs and cheatsheet
[/] I have not broken the cheatsheet

AndreasArvidsson commented 2 months ago

Totally in agreements there