-
Notifications
You must be signed in to change notification settings - Fork 7
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
Breaks build when used with CKEditor5 styles #34
Comments
After digging a bit, I can see that part of the issue is caused by
{
postcssPlugin: 'postcss-ckeditor5-theme-importer',
Once( root, { result } ) {
// Clone the options, don't alter the original options object.
const options = Object.assign( {}, pluginOptions, {
debug: pluginOptions.debug || false,
postCssOptions: {
plugins: [
require( 'postcss-import' )(),
require( './themelogger' )()
]
},
root, result
} );
return importThemeFile( options );
}
} From what I can make out the plugins themselves are provided and should not be imported via This approach was functioning until |
We're trying to update CKEditor5 to v34 in our Nuxt apps. One of the major changes was a switch to PostCSS 8. Following the instructions for this package breaks our build in a peculiar way.
We had it working working with CKEditor5 v33 and PostCSS 7. Our current version of Nuxt is
2.15.8
.The only difference between our previous and current config is the addition of
@nuxt/postcss8
inbuildModules
. Here is the relevant part in our currentnuxt.config.ts
:The error output when we run
nuxt build
:If
"@nuxt/postcss8"
or the postcss configurationpostcss: styles.getPostCssConfig({
are removed the error disappears.The only changes to the dependancies are
@nuxt/postcss8
and the various CKEditor5 packages updated to latest. May be worth noting that the exact same error occurs with CKEditor5 v33 as soon as@nuxt/postcss8
is added.The text was updated successfully, but these errors were encountered: