search

biojs / slush-biojs

DEPRECATED - please use https://github.com/biojs/generator-biojs-webcomponents
https://github.com/biojs/generator-biojs-webcomponents
7 stars 3 forks source link

gulp issues on windows #10

Closed wilzbach closed 9 years ago

wilzbach commented 9 years ago

@ljgarcia reported some issues with the generated gulpfile on windows.

Some of the gulp tasks use a slash and thus probably don't work properly on windows (e.g. gulp doesn't find any tests).

ljgarcia commented 9 years ago

On 03/12/2014 15:31, Seb wrote:

@ljgarcia https://github.com/ljgarcia reported some issues with the generated gulpfile on windows.

Some of the gulp tasks use a slash and thus probably don't work properly on windows (e.g. gulp doesn't find any tests).

— Reply to this email directly or view it on GitHub https://github.com/biojs/slush-biojs/issues/10.

Actually, I do not think it is due to slashes. I think slashes would be a problem only if you have spaces in between, do you? For instance "temp/mydir" works fine but "temp/my dir" might be tricky on Windows. I think you would need to add quotation marks around "my dir".

I will try in a different Windows machine, similar configuration. It could be just this machine. Will let you know how it goes.

Thanks!

wilzbach commented 9 years ago

Hi @ljgarcia,

I did a couple of changes to slush in the last days.

The major change is that no gulpfile is created by default - I am still trying to reduce the complexity.

Maybe the changes fixed your problem on windows ;-)

ljgarcia commented 9 years ago

On 14/12/2014 19:52, Seb wrote:

Hi @ljgarcia https://github.com/ljgarcia,

I did a couple of changes https://github.com/biojs/slush-biojs/compare/56434b7c8bbaf2abd52dbd79fc4811d8d31b9c47...master to slush in the last days.

The major change is that no gulpfile is created by default - I am still trying to reduce the complexity.

Maybe the changes fixed your problem on windows ;-)

— Reply to this email directly or view it on GitHub https://github.com/biojs/slush-biojs/issues/10#issuecomment-66926856.

Hi Seb,

Thanks. Sorry for my late response.

So, I tried it, even with the gulpfile, and it worked fine, as far as I do not use jshint. Tests run fine. If I use jshint, it tells me that the version 1.8.4 is not compatible. It displays the possible options and 1.8.4 is not in the list. I double checked with npm install -g slush-biojs, and indeed 1.8.4 is not listed. What jshint version are you installing? Would it be the latest one or a particular version?

Regards, Leyla

wilzbach commented 9 years ago

If I use jshint, it tells me that the version 1.8.4 is not compatible.

Yes this was a bug - thanks for reporting :) (I uploaded a new slush.)

What jshint version are you installing?

The version for the dependencies are set here.

Cheers,

Seb

ljgarcia commented 9 years ago

On 17/12/2014 16:00, Seb wrote:

If I use jshint, it tells me that the version 1.8.4 is not compatible.

Yes this was a bug - thanks for reporting :) (I uploaded a new slush https://github.com/biojs/slush-biojs/commit/5adfe90e519907bba1ccd3b7a46676d95ea65c6a.)

What jshint version are you installing?

The version for the dependencies are set here https://github.com/biojs/slush-biojs/blob/master/inquirer.js#L261.

Cheers,

Seb

— Reply to this email directly or view it on GitHub https://github.com/biojs/slush-biojs/issues/10#issuecomment-67343780.

Yeah! It is all working now!

Thanks Seb!