search

nwolverson / purescript-language-server

MIT License
183 stars 41 forks source link

Detect new-style spago workspaces #188

Closed f-f closed 1 year ago

andys8 commented 1 year ago

@f-f Will spago.yml also be a thing or will it be wrong?

f-f commented 1 year ago

We don't plan to support spago.yml at the moment.

andys8 commented 1 year ago

Sounds good. I think it's better for consistency to not support the variant. It could be a distinct, very specific error message in spago in case spago.yaml is missing but spago.yml exists :)

f-f commented 1 year ago

@andys8 it's a good idea. I don't have bandwidth to take this on right now, but I'd welcome a PR for this 🙂