`index.jade` file fails to compile in `npm run dev` #385

thenickcox · 2015-03-30T21:16:38Z

On node 0.10.16:

nick@navicans-MacBook-Pro ~/git/marionettejs.com [master]
± % npm run dev                                                                                                                                                                                                                                                          !10571

> [email protected] dev /Users/nick/git/marionettejs.com
> grunt dev

Running "clean:dist" (clean) task
>> 1 path cleaned.

Running "connect:server" (connect) task
Started connect web server on http://0.0.0.0:8000

... (a bunch of jshint-related output)

Running "jade:compile" (jade) task
>> Error: src/sections/_header.jade:1
>>   > 1| include _nav
>>     2| .masthead
>>     3|   .wrapper
>>     4|     .left
>>
>> ENOENT, no such file or directory 'dist/js/classList.js'
Warning: Jade failed to compile "src/index.jade". Use --force to continue.

Aborted due to warnings.
Running "notify:preHTML" (notify) task

Running "gitty:releaseTag:marionette" (gitty:releaseTag) task
>> Release Tag! -- v2.4.1

Running "jade:compile" (jade) task
>> Error: src/sections/_header.jade:1
>>   > 1| include _nav
>>     2| .masthead
>>     3|   .wrapper
>>     4|     .left
>>
>> ENOENT, no such file or directory 'dist/js/classList.js'
Warning: Jade failed to compile "src/index.jade". Use --force to continue.

Aborted due to warnings.
Completed in 90.605s at Mon Mar 30 2015 14:14:49 GMT-0700 (PDT) - Waiting...
^C%

peterblazejewicz · 2015-03-30T21:29:00Z

dist/js/classList.js is created during build process, it's part of inlined JavaScript code. Do you receive any JSHint errors before Jade task fails?

thenickcox · 2015-03-30T21:37:46Z

Guess I shouldn't have abbreviated that part of the output! Here's grunt jscs:grunt on its own:

± % grunt jscs:grunt                                                                                                                                                                                                                                                     !10645
Running "jscs:grunt" (jscs) task
Expected indentation of 8 characters at gruntfile.js :
    35 |      },
    36 |      home: {
    37 |          options: {
----------------^
    38 |              css: [
    39 |                  'dist/styles/marionette.css'
Expected indentation of 10 characters at gruntfile.js :
    36 |      home: {
    37 |          options: {
    38 |              css: [
------------------^
    39 |                  'dist/styles/marionette.css'
    40 |              ]
Expected indentation of 8 characters at gruntfile.js :
    39 |                  'dist/styles/marionette.css'
    40 |              ]
    41 |          },
----------------^
    42 |          src: 'dist/index.html',
    43 |          dest: 'dist/index.html'
Expected indentation of 8 characters at gruntfile.js :
    40 |              ]
    41 |          },
    42 |          src: 'dist/index.html',
----------------^
    43 |          dest: 'dist/index.html'
    44 |      },
Expected indentation of 8 characters at gruntfile.js :
    41 |          },
    42 |          src: 'dist/index.html',
    43 |          dest: 'dist/index.html'
----------------^
    44 |      },
    45 |      inspector: {
Expected indentation of 8 characters at gruntfile.js :
    44 |      },
    45 |      inspector: {
    46 |          options: {
----------------^
    47 |            css: [
    48 |              'dist/styles/inspector.css'
Expected indentation of 8 characters at gruntfile.js :
    49 |            ]
    50 |          },
    51 |          src: 'dist/inspector/index.html',
----------------^
    52 |          dest: 'dist/inspector/index.html'
    53 |      }
Expected indentation of 8 characters at gruntfile.js :
    50 |          },
    51 |          src: 'dist/inspector/index.html',
    52 |          dest: 'dist/inspector/index.html'
----------------^
    53 |      }
    54 |    },
>> 8 code style errors found!
Warning: Task "jscs:grunt" failed. Use --force to continue.

Aborted due to warnings.

thenickcox · 2015-03-30T21:44:37Z

I thought maybe running this with --force would ignore the errors, but it still aborts.

peterblazejewicz · 2015-03-30T21:44:51Z

For some weird reason your local branch seems to be out of sync of master branch:

➜  marionettejs.com git:(master) grunt jscs:grunt
Running "jscs:grunt" (jscs) task
>> 1 files without code style errors.

Done, without errors.

thenickcox · 2015-03-30T21:46:52Z

Could there be something else wrong?

± % git pull origin master                                                                                                                                                                                                                                               !10649
From https://github.com/marionettejs/marionettejs.com
 * branch            master     -> FETCH_HEAD
Already up-to-date.

nick@navicans-MacBook-Pro ~/git/marionettejs.com [master]
± % grunt jscs:grunt                                                                                                                                                                                                                                                     !10650
Running "jscs:grunt" (jscs) task
Expected indentation of 8 characters at gruntfile.js :
    35 |      },
    36 |      home: {
    37 |          options: {
----------------^
    38 |              css: [
    39 |                  'dist/styles/marionette.css'
Expected indentation of 10 characters at gruntfile.js :
    36 |      home: {
    37 |          options: {
    38 |              css: [
------------------^
    39 |                  'dist/styles/marionette.css'
    40 |              ]
Expected indentation of 8 characters at gruntfile.js :
    39 |                  'dist/styles/marionette.css'
    40 |              ]
    41 |          },
----------------^
    42 |          src: 'dist/index.html',
    43 |          dest: 'dist/index.html'
Expected indentation of 8 characters at gruntfile.js :
    40 |              ]
    41 |          },
    42 |          src: 'dist/index.html',
----------------^
    43 |          dest: 'dist/index.html'
    44 |      },
Expected indentation of 8 characters at gruntfile.js :
    41 |          },
    42 |          src: 'dist/index.html',
    43 |          dest: 'dist/index.html'
----------------^
    44 |      },
    45 |      inspector: {
Expected indentation of 8 characters at gruntfile.js :
    44 |      },
    45 |      inspector: {
    46 |          options: {
----------------^
    47 |            css: [
    48 |              'dist/styles/inspector.css'
Expected indentation of 8 characters at gruntfile.js :
    49 |            ]
    50 |          },
    51 |          src: 'dist/inspector/index.html',
----------------^
    52 |          dest: 'dist/inspector/index.html'
    53 |      }
Expected indentation of 8 characters at gruntfile.js :
    50 |          },
    51 |          src: 'dist/inspector/index.html',
    52 |          dest: 'dist/inspector/index.html'
----------------^
    53 |      }
    54 |    },
>> 8 code style errors found!
Warning: Task "jscs:grunt" failed. Use --force to continue.

Aborted due to warnings.

Fix code linting after deps update. Closes #385 Verified locally as Travis task failed again.

peterblazejewicz · 2015-03-30T22:15:03Z

@thenickcox fixed and merged. If you fetch update and rebase, everything should be ok.
Thanks!

thenickcox · 2015-03-30T22:16:48Z

@peterblazejewicz Worked like a charm!

± % grunt jscs:grunt                                                                                                                                                                                                                                                     !10662
Running "jscs:grunt" (jscs) task
>> 1 files without code style errors.

Done, without errors.

thenickcox mentioned this issue Mar 30, 2015

Line breaks in the markdown docs are translated as <br /> tags erroneously. #384

Open

peterblazejewicz mentioned this issue Mar 30, 2015

Fix code linting after deps update. Closes #385 #386

Merged

peterblazejewicz closed this as completed in c5f3596 Mar 30, 2015

peterblazejewicz added a commit that referenced this issue Mar 30, 2015

Merge pull request #386 from peterblazejewicz/fix/385

0b47ade

Fix code linting after deps update. Closes #385 Verified locally as Travis task failed again.

thenickcox pushed a commit to thenickcox/marionettejs.com that referenced this issue Mar 31, 2015

Fix code linting after deps update. Closes marionettejs#385

2ae8da4

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

`index.jade` file fails to compile in `npm run dev` #385

`index.jade` file fails to compile in `npm run dev` #385

thenickcox commented Mar 30, 2015

peterblazejewicz commented Mar 30, 2015

thenickcox commented Mar 30, 2015

thenickcox commented Mar 30, 2015

peterblazejewicz commented Mar 30, 2015

thenickcox commented Mar 30, 2015

peterblazejewicz commented Mar 30, 2015

thenickcox commented Mar 30, 2015

index.jade file fails to compile in npm run dev #385

index.jade file fails to compile in npm run dev #385

Comments

thenickcox commented Mar 30, 2015

peterblazejewicz commented Mar 30, 2015

thenickcox commented Mar 30, 2015

thenickcox commented Mar 30, 2015

peterblazejewicz commented Mar 30, 2015

thenickcox commented Mar 30, 2015

peterblazejewicz commented Mar 30, 2015

thenickcox commented Mar 30, 2015

`index.jade` file fails to compile in `npm run dev` #385

`index.jade` file fails to compile in `npm run dev` #385