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

Use utils.getOptions() to get loader options #49

Open
wants to merge 1 commit into
base: 3.x
from

Conversation

@TacticalCode
Copy link

@TacticalCode TacticalCode commented Jun 21, 2020

Drop manually reading/parsing the options passed through Loader API in favor for the getOptions method already provided by loader-utils, soon to be integrated into webpack5 itself.

Tested and working with both inline query and options object, using only minimal htmlminify options as shown here: https://github.com/bazilio91/ejs-compiled-loader#htmlminify

Note: I did not test any other options. Potential problems if "merge" does not create a "deep copy" of options since it's returned read-only by getOptions()!

Drop manually reading/parsing the options passed through Loader API in favor for the getOptions method already provided by loader-utils, soon to be integrated into webpack5 itself.

Tested and working with both inline query and options object, using only minimal htmlminify options as shown here: https://github.com/bazilio91/ejs-compiled-loader#htmlminify
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.