From 8f43be567f0118df6dec0bc01ecb829be37ba209 Mon Sep 17 00:00:00 2001
From: ULIVZ <472590061@qq.com>
Date: Mon, 4 Feb 2019 02:02:44 +0800
Subject: [PATCH] refactor: update code style
---
.eslintrc.js | 3 ++-
packages/@vuepress/core/lib/app/app.js | 4 +--
packages/@vuepress/core/lib/dev.js | 4 +--
.../frontmatterBlock/loader.js | 10 +++----
.../@vuepress/core/lib/plugin-api/index.js | 4 +--
.../override/EnhanceAppFilesOption.js | 4 +--
.../@vuepress/core/lib/plugin-api/util.js | 8 +++---
.../@vuepress/core/lib/prepare/AppContext.js | 4 +--
.../@vuepress/core/lib/prepare/CacheLoader.js | 14 +++++-----
.../core/lib/prepare/ClientComputedMixin.js | 4 +--
.../@vuepress/core/lib/prepare/loadTheme.js | 12 ++++-----
.../core/lib/webpack/DevLogPlugin.js | 6 ++---
packages/@vuepress/markdown-loader/index.js | 24 ++++++++---------
.../@vuepress/markdown/lib/highlightLines.js | 4 +--
.../@vuepress/markdown/lib/lineNumbers.js | 4 +--
packages/@vuepress/markdown/lib/preWrapper.js | 4 +--
.../clientRootMixin.js | 6 ++---
.../plugin-back-to-top/BackToTop.vue | 6 ++---
.../plugin-pwa/lib/enhanceAppFile.js | 6 ++---
.../@vuepress/plugin-search/SearchBox.vue | 10 +++----
.../shared-utils/scripts/update-index.js | 14 +++++-----
.../theme-default/components/Navbar.vue | 4 +--
.../theme-default/components/Page.vue | 26 +++++++++----------
.../theme-default/components/SidebarLink.vue | 12 ++++-----
packages/@vuepress/theme-default/index.js | 4 +--
.../theme-default/layouts/Layout.vue | 20 +++++++-------
.../theme-vue/components/CarbonAds.vue | 4 +--
packages/vuepress/lib/checkEnv.js | 6 ++---
packages/vuepress/lib/handleUnknownCommand.js | 4 +--
29 files changed, 118 insertions(+), 117 deletions(-)
diff --git a/.eslintrc.js b/.eslintrc.js
index a694c834..0d87945d 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -6,6 +6,7 @@ module.exports = {
],
rules: {
indent: ['error', 2, { MemberExpression: 'off' }],
- "no-undef": ["error"]
+ "no-undef": ["error"],
+ 'operator-linebreak': ["error", "before"]
}
}
diff --git a/packages/@vuepress/core/lib/app/app.js b/packages/@vuepress/core/lib/app/app.js
index 9adb77c0..1adc33e3 100644
--- a/packages/@vuepress/core/lib/app/app.js
+++ b/packages/@vuepress/core/lib/app/app.js
@@ -21,8 +21,8 @@ if (module.hot) {
module.hot.accept(VUEPRESS_TEMP_PATH + '/internal/siteData.js', () => {
if (siteData.base !== prevBase) {
window.alert(
- `[vuepress] Site base has changed. ` +
- `Please restart dev server to ensure correct asset paths.`
+ `[vuepress] Site base has changed. `
+ + `Please restart dev server to ensure correct asset paths.`
)
}
})
diff --git a/packages/@vuepress/core/lib/dev.js b/packages/@vuepress/core/lib/dev.js
index 6c683815..c8051d39 100644
--- a/packages/@vuepress/core/lib/dev.js
+++ b/packages/@vuepress/core/lib/dev.js
@@ -101,8 +101,8 @@ async function prepareServer (sourceDir, cliOptions = {}, context) {
const { host, displayHost } = await resolveHost(cliOptions.host || ctx.siteConfig.host)
// debug in a running dev process.
- process.stdin &&
- process.stdin.on('data', chunk => {
+ process.stdin
+ && process.stdin.on('data', chunk => {
const parsed = chunk.toString('utf-8').trim()
if (parsed === '*') {
console.log(Object.keys(ctx))
diff --git a/packages/@vuepress/core/lib/internal-plugins/frontmatterBlock/loader.js b/packages/@vuepress/core/lib/internal-plugins/frontmatterBlock/loader.js
index bb29bece..c36c6fd3 100644
--- a/packages/@vuepress/core/lib/internal-plugins/frontmatterBlock/loader.js
+++ b/packages/@vuepress/core/lib/internal-plugins/frontmatterBlock/loader.js
@@ -12,11 +12,11 @@ module.exports = function (source, map) {
const cached = cache.get(file)
const parsed = parseStrippedFrontmatter(source)
- if (cached &&
- cached.data &&
- parsed &&
- parsed.data &&
- JSON.stringify(cached.data) !== JSON.stringify(parsed.data)
+ if (cached
+ && cached.data
+ && parsed
+ && parsed.data
+ && JSON.stringify(cached.data) !== JSON.stringify(parsed.data)
) {
frontmatterEmitter.emit('update')
}
diff --git a/packages/@vuepress/core/lib/plugin-api/index.js b/packages/@vuepress/core/lib/plugin-api/index.js
index 69649f80..83639fb5 100644
--- a/packages/@vuepress/core/lib/plugin-api/index.js
+++ b/packages/@vuepress/core/lib/plugin-api/index.js
@@ -169,8 +169,8 @@ module.exports = class PluginAPI {
this.options[option.name].add(pluginName, value)
} else if (value !== undefined) {
logger.warn(
- `${chalk.gray(pluginName)} ` +
- `Invalid value for "option" ${chalk.cyan(option.name)}: ${warnMsg}`
+ `${chalk.gray(pluginName)} `
+ + `Invalid value for "option" ${chalk.cyan(option.name)}: ${warnMsg}`
)
}
return this
diff --git a/packages/@vuepress/core/lib/plugin-api/override/EnhanceAppFilesOption.js b/packages/@vuepress/core/lib/plugin-api/override/EnhanceAppFilesOption.js
index d9446d5c..c109c130 100644
--- a/packages/@vuepress/core/lib/plugin-api/override/EnhanceAppFilesOption.js
+++ b/packages/@vuepress/core/lib/plugin-api/override/EnhanceAppFilesOption.js
@@ -67,8 +67,8 @@ module.exports = class EnhanceAppFilesOption extends AsyncOption {
}
} else {
logger.developer(
- chalk.gray(`[${pluginName}] `) +
- `${chalk.cyan(enhanceAppFile)} Not Found.`
+ chalk.gray(`[${pluginName}] `)
+ + `${chalk.cyan(enhanceAppFile)} Not Found.`
)
}
}
diff --git a/packages/@vuepress/core/lib/plugin-api/util.js b/packages/@vuepress/core/lib/plugin-api/util.js
index bf40a15c..706da428 100644
--- a/packages/@vuepress/core/lib/plugin-api/util.js
+++ b/packages/@vuepress/core/lib/plugin-api/util.js
@@ -26,8 +26,8 @@ exports.flattenPlugin = function (
if (!valid) {
if (pluginOptions !== undefined) {
logger.warn(
- `[${chalk.gray(shortcut)}] ` +
- `Invalid value for "pluginOptions" ${chalk.cyan(name)}: ${warnMsg}`
+ `[${chalk.gray(shortcut)}] `
+ + `Invalid value for "pluginOptions" ${chalk.cyan(name)}: ${warnMsg}`
)
}
pluginOptions = {}
@@ -69,8 +69,8 @@ exports.normalizePluginsConfig = function (pluginsConfig) {
if (!valid) {
if (pluginsConfig !== undefined) {
logger.warn(
- `[${chalk.gray('config')}] ` +
- `Invalid value for "plugin" field : ${warnMsg}`
+ `[${chalk.gray('config')}] `
+ + `Invalid value for "plugin" field : ${warnMsg}`
)
}
pluginsConfig = []
diff --git a/packages/@vuepress/core/lib/prepare/AppContext.js b/packages/@vuepress/core/lib/prepare/AppContext.js
index c1176a15..09aee105 100644
--- a/packages/@vuepress/core/lib/prepare/AppContext.js
+++ b/packages/@vuepress/core/lib/prepare/AppContext.js
@@ -130,8 +130,8 @@ module.exports = class AppContext {
const siteConfig = this.siteConfig
const shouldUseLastUpdated = (
- themeConfig.lastUpdated ||
- Object.keys(siteConfig.locales && themeConfig.locales || {})
+ themeConfig.lastUpdated
+ || Object.keys(siteConfig.locales && themeConfig.locales || {})
.some(base => themeConfig.locales[base].lastUpdated)
)
diff --git a/packages/@vuepress/core/lib/prepare/CacheLoader.js b/packages/@vuepress/core/lib/prepare/CacheLoader.js
index 7b611a3a..7d102914 100644
--- a/packages/@vuepress/core/lib/prepare/CacheLoader.js
+++ b/packages/@vuepress/core/lib/prepare/CacheLoader.js
@@ -38,19 +38,19 @@ exports.getCacheLoaderOptions = function (siteConfig, cliOptions, cwd, isProd) {
siteConfig.markdown
? JSON.stringify(siteConfig.markdown)
: ''
- ) +
- (
+ )
+ + (
siteConfig.markdown && siteConfig.markdown.extendMarkdown
? siteConfig.markdown.extendMarkdown.toString()
: ''
- ) +
- (
+ )
+ + (
siteConfig.extendMarkdown
? siteConfig.extendMarkdown.toString()
: ''
- ) +
- (siteConfig.chainWebpack || '').toString() +
- (siteConfig.configureWebpack || '').toString()
+ )
+ + (siteConfig.chainWebpack || '').toString()
+ + (siteConfig.configureWebpack || '').toString()
)
})
diff --git a/packages/@vuepress/core/lib/prepare/ClientComputedMixin.js b/packages/@vuepress/core/lib/prepare/ClientComputedMixin.js
index 2e99ba4c..dbe8a3f7 100644
--- a/packages/@vuepress/core/lib/prepare/ClientComputedMixin.js
+++ b/packages/@vuepress/core/lib/prepare/ClientComputedMixin.js
@@ -74,8 +74,8 @@ module.exports = siteData => {
const siteTitle = this.$siteTitle
const selfTitle = page.frontmatter.home ? null : (
- page.frontmatter.title || // explicit title
- page.title // inferred title
+ page.frontmatter.title // explicit title
+ || page.title // inferred title
)
return siteTitle
? selfTitle
diff --git a/packages/@vuepress/core/lib/prepare/loadTheme.js b/packages/@vuepress/core/lib/prepare/loadTheme.js
index dcede9eb..43056f73 100644
--- a/packages/@vuepress/core/lib/prepare/loadTheme.js
+++ b/packages/@vuepress/core/lib/prepare/loadTheme.js
@@ -34,10 +34,10 @@ module.exports = async function loadTheme (ctx) {
const themeResolver = getThemeResolver()
const localThemePath = path.resolve(vuepressDir, 'theme')
- const useLocalTheme =
- !fs.existsSync(theme) &&
- fs.existsSync(localThemePath) &&
- (fs.readdirSync(localThemePath)).length > 0
+ const useLocalTheme
+ = !fs.existsSync(theme)
+ && fs.existsSync(localThemePath)
+ && (fs.readdirSync(localThemePath)).length > 0
let themePath = null // Mandatory
let themeEntryFile = null // Optional
@@ -119,8 +119,8 @@ module.exports = async function loadTheme (ctx) {
const readdirSync = dir => fs.existsSync(dir) && fs.readdirSync(dir) || []
// built-in named layout or not.
- const isInternal = componentName => componentName === 'Layout' ||
- componentName === 'NotFound'
+ const isInternal = componentName => componentName === 'Layout'
+ || componentName === 'NotFound'
const layoutComponentMap = layoutDirs
.map(
diff --git a/packages/@vuepress/core/lib/webpack/DevLogPlugin.js b/packages/@vuepress/core/lib/webpack/DevLogPlugin.js
index 15a610d2..b51927e6 100644
--- a/packages/@vuepress/core/lib/webpack/DevLogPlugin.js
+++ b/packages/@vuepress/core/lib/webpack/DevLogPlugin.js
@@ -25,9 +25,9 @@ module.exports = class DevLogPlugin {
const displayUrl = `http://${displayHost}:${port}${publicPath}`
logger.success(
- `${chalk.gray(`[${time}]`)} Build ${chalk.italic(stats.hash.slice(0, 6))} ` +
- `finished in ${stats.endTime - stats.startTime} ms! ` +
- (
+ `${chalk.gray(`[${time}]`)} Build ${chalk.italic(stats.hash.slice(0, 6))} `
+ + `finished in ${stats.endTime - stats.startTime} ms! `
+ + (
isFirst
? ''
: `${chalk.gray(`(${displayUrl})`)}`
diff --git a/packages/@vuepress/markdown-loader/index.js b/packages/@vuepress/markdown-loader/index.js
index 2043fc9d..8e53a7e2 100644
--- a/packages/@vuepress/markdown-loader/index.js
+++ b/packages/@vuepress/markdown-loader/index.js
@@ -49,9 +49,9 @@ module.exports = function (src) {
// returned component, changes in frontmatter do not trigger proper updates
const cachedData = devCache.get(file)
if (cachedData && (
- cachedData.inferredTitle !== inferredTitle ||
- JSON.stringify(cachedData.frontmatterData) !== JSON.stringify(frontmatter.data) ||
- headersChanged(cachedData.headers, headers)
+ cachedData.inferredTitle !== inferredTitle
+ || JSON.stringify(cachedData.frontmatterData) !== JSON.stringify(frontmatter.data)
+ || headersChanged(cachedData.headers, headers)
)) {
// frontmatter changed... need to do a full reload
module.exports.frontmatterEmitter.emit('update')
@@ -91,19 +91,19 @@ module.exports = function (src) {
if (!fs.existsSync(file) && (!altfile || !fs.existsSync(altfile))) {
this.emitWarning(
new Error(
- `\nFile for relative link "${link}" does not exist.\n` +
- `(Resolved file: ${file})\n`
+ `\nFile for relative link "${link}" does not exist.\n`
+ + `(Resolved file: ${file})\n`
)
)
}
})
const res = (
- `\n` +
- `
'
}).join('')
- const highlightLinesWrapperCode =
- `