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

[v3] Support sass-loader v8 #4662

Merged
merged 4 commits into from Oct 10, 2019
Merged

Conversation

sodatea
Copy link
Member

@sodatea sodatea commented Oct 9, 2019

The warning on the loader version was removed to avoid frustrations.
Users could postpone the sass-loader upgrade until they decide to move to Vue CLI v4.

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Docs
  • Underlying tools
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

Other information:

@sodatea sodatea merged commit 02859cf into vuejs:v3 Oct 10, 2019
@sodatea sodatea deleted the feat-v3-sass-loader-compat branch October 10, 2019 01:59
@ghulamhussain
Copy link

ghulamhussain commented Oct 16, 2019

@sodatea This has created a new bug in v 3.12 when adding vuetify plugin using vue add vetify
`conflicting versions for project dependency "sass-loader":

  • ^8.0.0 injected by generator "undefined"
  • ^7.1.0 injected by generator "vue-cli-plugin-vuetify"

Using newer version (^8.0.0), but this may cause build errors.
`

@sodatea
Copy link
Member Author

sodatea commented Oct 16, 2019

It's a friendly warning but not a bug. The undefined message could be improved though.

@ghulamhussain
Copy link

i am having this error in when i added vuetify

Failed to compile.

./node_modules/vuetify/src/styles/components/_selection-controls.sass (./node_modules/css-loader??ref--9-oneOf-3-1!./node_modules/postcss-loader/src??ref--9-oneOf-3-2!./node_modules/sass-loader/dist/cjs.js??ref--9-oneOf-3-3!./node_modules/vuetify/src/styles/components/_selection-controls.sass)
Module build failed (from ./node_modules/sass-loader/dist/cjs.js):
ValidationError: Invalid options object. Sass Loader has been initialised using an options object that does not match the API schema.

  • options has an unknown property 'data'. These properties are valid:
    object { implementation?, sassOptions?, prependData?, sourceMap?, webpackImporter? }
    at validate (/home/hussaghu/Practice/vuetest/node_modules/sass-loader/node_modules/schema-utils/dist/validate.js:50:11)
    at Object.loader (/home/hussaghu/Practice/vuetest/node_modules/sass-loader/dist/index.js:36:28)

@sodatea
Copy link
Member Author

sodatea commented Oct 16, 2019

Well, seems vuetify plugin hasn't support sass-loader 8 yet. Then downgrade the dependency in your package.json to 7.1.0.

@ghulamhussain
Copy link

@sodatea thanks the problem is in vuetify plugin that doesn't support sass loader.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants