Skip to content

Conversation

@alexander-akait
Copy link
Member

BREAKING CHANGE: move all sass (includePaths, importer, functions) options to the sassOptions option. The functions option can't be used as Function, you should use sassOption as Function to achieve this.

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

move all sass options to the sassOptions option

Breaking Changes

Yes

All sass (includePaths, importer, functions) options now in the sassOptions option. The functions option can't be used as Function, you should use sassOption as Function to achieve this.

For example:

sassOptions: (loaderContext){ 
  return { functions: getSassFunctions(loaderContext); };
}

Additional Info

No

BREAKING CHANGE: move all sass (`includePaths`, `importer`, `functions`) options to the `sassOptions` option. The `functions` option can't be used as `Function`, you should use `sassOption` as `Function` to achieve this.
@alexander-akait alexander-akait force-pushed the refactor-sassOptions-option branch from 5712d50 to 951d089 Compare August 20, 2019 17:10
@alexander-akait alexander-akait merged commit 9c5028b into next Aug 20, 2019
@alexander-akait alexander-akait deleted the refactor-sassOptions-option branch August 20, 2019 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants