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

update kanso-utils and less #10

Closed
wants to merge 3 commits into from
Closed

update kanso-utils and less #10

wants to merge 3 commits into from

Conversation

thoka
Copy link

@thoka thoka commented Jan 18, 2016

This solves #9 for me.

@MidnightLightning
Copy link

Not sure why the node_modules folder is checked in on this repository in the first place; to fix, it should probably just be a change to the package.json file to update the version, and add the node_modules folder to the .gitignore listing, no?

@thoka
Copy link
Author

thoka commented Jan 18, 2016

@MidnightLightning hm, there is no package.json

I'm not very familiar how kanso packages work.
As I understand, kanso fetches its packages from its own repositories and will not use npm to get needed npm packages ?

@MidnightLightning
Copy link

Ah, you're right; it's a kanso.json file for Kanso packages, though it creates a node_modules to store dependencies in like NPM. So the way you've got it looks good 👍

@mandric mandric closed this Jul 13, 2017
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