Skip to content
This repository was archived by the owner on Dec 17, 2025. It is now read-only.

Default getOptions to return {}#164

Closed
sliftist wants to merge 1 commit intowebpack:masterfrom
sliftist:master
Closed

Default getOptions to return {}#164
sliftist wants to merge 1 commit intowebpack:masterfrom
sliftist:master

Conversation

@sliftist
Copy link

Default getOptions to return {}, that way loaders don't break when the loader options are removed. For example, if the loader changes from "example-loader?setFlag" to "example-loader".

Without this change every single user of getOptions will almost certainly want to default getOptions to null manually.

Default getOptions to return {}, that way loaders don't break when the loader options are removed. For example, if the loader changes from "example-loader?setFlag" to "example-loader".

Without this change every single user of getOptions will almost certainly want to default getOptions to null manually.
@jsf-clabot
Copy link

jsf-clabot commented Feb 22, 2020

CLA assistant check
All committers have signed the CLA.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants