at-import / SassyLists

MIT License
276 stars 27 forks source link

npm install fails on peerDependencies of Bootcamp #26

Closed leonderijke closed 10 years ago

leonderijke commented 10 years ago

After a fresh clone of the repository, issuing npm install results in:

npm ERR! peerinvalid The package grunt-contrib-sass does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer bootcamp@1.1.4 wants grunt-contrib-sass@~0.7.2

Apparently, Bootcamp updated their dependencies to a newer version of grunt-contrib-sass: thejameskyle/bootcamp@21be30e9c657c6a090689cb8d5a53aedccde44e7.

After manually updating SassyLists dependency on grunt-contrib-sass to ~0.7.2 in package.json, I get the following error when running grunt test:

Running "sass:test" (sass) task
Syntax error: List index is 1 but list has no items for `nth'
        on line 11 of /Users/leon/werkmap/SassyLists/node_modules/bootcamp/dist/utils/_stringify.scss, in `should'
        from line 8 of test/SassyLists/_chunk.scss, in `@content'
        from line 8 of /Users/leon/werkmap/SassyLists/node_modules/bootcamp/dist/core/mixins/_it.scss, in `it'
        from line 7 of test/SassyLists/_chunk.scss, in `@content'
        from line 14 of /Users/leon/werkmap/SassyLists/node_modules/bootcamp/dist/core/mixins/_describe.scss, in `describe'
        from line 1 of test/SassyLists/_chunk.scss
        from line 8 of ./test/test.scss
  Use --trace for backtrace.
Warning: Exited with error code 1 Use --force to continue.

Aborted due to warnings.

Do you have any ideas for how to fix this?

KittyGiraudel commented 10 years ago

@fweinb, any idea perhaps?

FWeinb commented 10 years ago

Will have a look at this tomorrow.

leonderijke commented 10 years ago

As far as I can tell, we're dealing with two issues here:

I'll open a separate issue for the second one and will try to debug further. Edit: #27.

KittyGiraudel commented 10 years ago

@TheJamesKyle Hey buddy, do you have some insight perhaps?

leonderijke commented 10 years ago

For the record, this issue is only about the failing npm install because of the peerDependencies. Fixing it is probably as easy as updating our grunt-contrib-sass dependency.

jamiebuilds commented 10 years ago

@HugoGiraudel If you don't want to upgrade you could always use 1.1.3

KittyGiraudel commented 10 years ago

Upgrade to... 2.0? Is it ready yet?

jamiebuilds commented 10 years ago

@HugoGiraudel heh... I could do a pre-release?

KittyGiraudel commented 10 years ago

I'd use it.

KittyGiraudel commented 10 years ago

Done.