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

Add explanation about npm packages #99

Merged
merged 2 commits into from
Feb 6, 2019
Merged

Add explanation about npm packages #99

merged 2 commits into from
Feb 6, 2019

Conversation

f-f
Copy link
Member

@f-f f-f commented Feb 6, 2019

Explain why my proposal here doesn't work:
https://discourse.purescript.org/t/post-purescript-package-management-explained/615/7

This thread is also highly informative on the matter:
purescript/purescript#631 (comment)

@f-f f-f requested a review from justinwoo February 6, 2019 17:54
@chexxor
Copy link

chexxor commented Feb 6, 2019

Nice explanation. I believe it's likely that this won't prevent/end future requests for spago to integrate with other build/dep management tools, but this at least gives a common starting point for those discussions. I think it's possible there's still some ways to support these other tools in a limited scope, so perhaps those future discussions will have proposals for them.

If you'd like to more absolutely block future requests for support for other dep tools, I suggest putting a more pointed sentence in there to state so.

README.md Outdated Show resolved Hide resolved
@f-f
Copy link
Member Author

f-f commented Feb 6, 2019

@chexxor: integrating with other tools is hard and it's a lot of work, but it's an actual UX issue.
I would love myself to not have to use npm and yarn and lerna and whatever else. Setting up the JS+PS build for our monorepo at work was very painful.

So I wouldn't like to exclude this possibility, as someone might come up with a clever solution and PR it 😄

as suggested by @JordanMartinez
@f-f f-f merged commit d66ac73 into master Feb 6, 2019
@f-f f-f deleted the f-f/npm-faq branch February 7, 2019 23:10
elliotdavies pushed a commit to elliotdavies/spago that referenced this pull request Jul 1, 2019
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

Successfully merging this pull request may close these issues.

3 participants