Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use pathy #1292

Closed
srghma opened this issue Oct 4, 2024 · 3 comments
Closed

use pathy #1292

srghma opened this issue Oct 4, 2024 · 3 comments

Comments

@srghma
Copy link
Contributor

srghma commented Oct 4, 2024

Suggestion: to improve type safety :)

  1. implement windows parser Can't parse windows style absolute paths purescript-contrib/purescript-pathy#19 (comment)
  2. finish https://github.com/srghma/purescript-pathy-node-fs/blob/main/src/Pathy/Node/FS/Common.purs
  3. use in spago
@srghma
Copy link
Contributor Author

srghma commented Oct 4, 2024

maybe use glob from purescript-node/purescript-node-fs#86

@srghma
Copy link
Contributor Author

srghma commented Oct 11, 2024

I have made a lib

https://github.com/srghma/purescript-pathy-node

but cannot publish since something something aff range wrong bower.json outdated

@f-f
Copy link
Member

f-f commented Nov 3, 2024

We're implementing stronger-typed paths in #1296 - these are custom types, made to fit the codebase, which should give the same assurances as using something like Pathy but better UX. I'll close this since I think that's the better approach, but we'll reopen it if we decide not to go with custom types

@f-f f-f closed this as completed Nov 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants