Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Vue/Style: Use passed style.test option as default instead of forcing css #1242

Merged
merged 1 commit into from
Dec 19, 2018

Conversation

timkelty
Copy link
Contributor

@timkelty timkelty commented Dec 7, 2018

neutrino => neutrino.use('@neutrinojs/vue', {
    style: {
          test: /\.s?css$/,
       }
})

The vue preset changes the default for style.modulesTest, but just defaults to css extensions.
If you are already overriding the test option, chances are you'll want it to use that as a default.

Note: in the past we've used Ramda.pathOr for this kind of nested accessor. It seemed like it had been mostly yanked from middlewares, so I did it the long way :)

@edmorley edmorley added this to the v9 milestone Dec 9, 2018
@edmorley edmorley added the bug label Dec 9, 2018
@edmorley edmorley merged commit d8560fa into neutrinojs:master Dec 19, 2018
timkelty added a commit to timkelty/neutrino that referenced this pull request Dec 19, 2018
If opts.style.test is passed, use that as a default, instead of forcing `.css`.
@timkelty timkelty deleted the vue-style-test-default branch December 19, 2018 19:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

2 participants