-
Notifications
You must be signed in to change notification settings - Fork 129
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
spago publish
command
#27
Comments
Doesn't work because Pursuit requires a whole lot of bower crap, and you can't actually develop libraries this way unless you intend for them to only work with package sets |
One day we won't need Bower to do library development, but that day isn't here yet |
Would Pursuit need to be changed to require less Bower stuff or can we shim it decently? |
It's such a giant ball of details I think you'd need to get people in #purescript to give you some spec and also inspect mostly Pursuit, but also Pulp. |
This is kind of out of scope since we'd need a package manager that actually works with version ranges afaict. But I wrote up some general information that's useful about how to work against Pursuit publishing, since sometimes the bower listing just stack overflows or OOMs or something: https://github.com/justinwoo/test-pulp-publish |
I agree that this is not going to be directly in our scope, since it's a lot about Pursuit itself. However, since Bower itself is deprecated, IMHO we should untangle the dependency on it (as an ecosystem) ASAP. So it's worth to look into this and eventually make Pursuit easier to deal with, so that this can then be implemented decently on our end of things PS: thanks for writing that down, good stuff 👏 |
"Deprecated" |
Update on the plan for this after reading the above summary by Justin and the source of |
Closing this in favor of #89 |
Spago is not yet ready for publishing package docs to pursuit. purescript/spago#27
We should have a
spago publish
command, that publishes the docs for the current package to Pursuit (aspulp publish
does)The text was updated successfully, but these errors were encountered: