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

Wrong dependencies #34

Open
Nek- opened this issue Jul 13, 2014 · 6 comments
Open

Wrong dependencies #34

Nek- opened this issue Jul 13, 2014 · 6 comments

Comments

@Nek-
Copy link

Nek- commented Jul 13, 2014

Why are they preprocessor as depedencies ?

I can understand that in order to test you could need some of theme, but it's exacly why require-dev exists.

Also notice that lessphp is not maintained anymore. The test should be use the standard less version using npm to install it.

@vendethiel
Copy link
Owner

maybe they should even be suggested. But I think at that point, none of them is maintained anymore ...

@Nek-
Copy link
Author

Nek- commented Jul 14, 2014

Why suggesting things we don't believe in ? ^^

They should be in dev dependencies if you need them for testing (even if it's better to have them as node dependencie for the dev), and that's all.

@vendethiel
Copy link
Owner

I mean something like "suggest-depend". Can't remember the name atm.

@Nek-
Copy link
Author

Nek- commented Jul 14, 2014

I understood, yep. My answer is still valid :p .

@vendethiel
Copy link
Owner

Yeah, though I created this library for people not having access to node stuff (hence why less/coffee are the PHP version), but if there's a really big difference, I should indeed use npm versions (and maybe create a pless, or lessp, of w/e)

@Nek-
Copy link
Author

Nek- commented Jul 15, 2014

i.e for less: bootstrap is not compabitle with lessphp.

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