search

tactivos / grunt-cdn

79 stars 53 forks source link

Arguments to path.join must be strings #10

Open sthomp opened 11 years ago

sthomp commented 11 years ago

I've copied the example provided in the README but keep getting this warning with no output. Is this a bug?

Running "cdn:dist" (cdn) task

cdn:html - ./dist/test.html Warning: Arguments to path.join must be strings Use --force to continue.

Aborted due to warnings.

sthomp commented 11 years ago

If I removed this line from the html file:

< link rel="stylesheet" type="text/css" href="/static/compiled.css?v=13512tyu3kds" />

Then it seems to work as the output is:

Running "cdn:dist" (cdn) task

cdn:html - dist/test.html

Done, without errors.

Seems like a bug parsing this line.

3dd13 commented 11 years ago

just had that problem and then checked out the source.

it actually requires the dest option, something like this:

  dist: {
      src: ['./dist/index.html'],
      dest: "./dist/"
  }
palominoz commented 10 years ago

I get the same error with HTMLs containing newlines in <code> blocks. Removing the newlines solves the issue, but it was not an acceptable solution for me

johanneswuerbach commented 10 years ago

I have the same problem with mailto links after upgrading from 0.2.3.

kaidez commented 10 years ago

Same problem for me...