Skip to content

Commit

Permalink
chore(builder): analyze mode warning rewording (#6924)
Browse files Browse the repository at this point in the history
  • Loading branch information
markhoney committed Feb 24, 2020
1 parent 1658702 commit 084a1ca
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion packages/builder/src/builder.js
Expand Up @@ -68,7 +68,7 @@ export default class Builder {

if (this.options.build.analyze) {
this.nuxt.hook('build:done', () => {
consola.warn('Notice: Please do not deploy bundles built with analyze mode, it\'s only for analyzing purpose.')
consola.warn('Notice: Please do not deploy bundles built with "analyze" mode, they\'re for analysis purposes only.')
})
}

Expand Down
2 changes: 1 addition & 1 deletion packages/builder/test/builder.ctor.test.js
Expand Up @@ -102,7 +102,7 @@ describe('builder: builder constructor', () => {
const doneHook = nuxt.hook.mock.calls[0][1]
doneHook()
expect(consola.warn).toBeCalledTimes(1)
expect(consola.warn).toBeCalledWith('Notice: Please do not deploy bundles built with analyze mode, it\'s only for analyzing purpose.')
expect(consola.warn).toBeCalledWith('Notice: Please do not deploy bundles built with "analyze" mode, they\'re for analysis purposes only.')
})

test('should support function template', () => {
Expand Down
2 changes: 1 addition & 1 deletion test/fixtures/with-config/with-config.test.js
Expand Up @@ -26,7 +26,7 @@ describe('with-config', () => {
}],
['Please use `build.postcss` in your nuxt.config.js instead of an external config file. Support for such files will be removed in Nuxt 3 as they remove all defaults set by Nuxt and can cause severe problems with features like alias resolving inside your CSS.'],
['Using styleResources without the @nuxtjs/style-resources is not suggested and can lead to severe performance issues.', 'Please use https://github.com/nuxt-community/style-resources-module'],
['Notice: Please do not deploy bundles built with analyze mode, it\'s only for analyzing purpose.']
['Notice: Please do not deploy bundles built with "analyze" mode, they\'re for analysis purposes only.']
])
expect(customCompressionMiddlewareFunctionName).toBe('damn')
}, hooks)
Expand Down

0 comments on commit 084a1ca

Please sign in to comment.