Skip to content

Commit

Permalink
refactor: typos (#4922)
Browse files Browse the repository at this point in the history
  • Loading branch information
manniL authored and pi0 committed Feb 1, 2019
1 parent 1fb9af3 commit 569b6aa
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 11 deletions.
12 changes: 6 additions & 6 deletions packages/builder/src/builder.js
Original file line number Diff line number Diff line change
Expand Up @@ -198,7 +198,7 @@ export default class Builder {
validateTemplate() {
// Validate template dependencies
const templateDependencies = this.template.dependencies
const dpendencyFixes = []
const dependencyFixes = []
for (const depName in templateDependencies) {
const depVersion = templateDependencies[depName]
const requiredVersion = `${depName}@${depVersion}`
Expand All @@ -209,24 +209,24 @@ export default class Builder {
const validVersion = semver.satisfies(pkg.version, depVersion)
if (!validVersion) {
consola.warn(`${requiredVersion} is required but ${depName}@${pkg.version} is installed!`)
dpendencyFixes.push(requiredVersion)
dependencyFixes.push(requiredVersion)
}
} else {
consola.warn(`${depName}@${depVersion} is required but not installed!`)
dpendencyFixes.push(requiredVersion)
dependencyFixes.push(requiredVersion)
}
}

// Suggest dependency fixes (TODO: automate me)
if (dpendencyFixes.length) {
if (dependencyFixes.length) {
consola.error(
'Please install missing dependencies:\n',
'\n',
'Using yarn:\n',
`yarn add ${dpendencyFixes.join(' ')}\n`,
`yarn add ${dependencyFixes.join(' ')}\n`,
'\n',
'Using npm:\n',
`npm i ${dpendencyFixes.join(' ')}\n`
`npm i ${dependencyFixes.join(' ')}\n`
)
throw new Error('Missing Template Dependencies')
}
Expand Down
9 changes: 6 additions & 3 deletions packages/core/src/resolver.js
Original file line number Diff line number Diff line change
Expand Up @@ -111,11 +111,14 @@ export default class Resolver {
throw new Error(`Cannot resolve "${path}" from "${resolvedPath}"`)
}

requireModule(path, { esm, useESM = esm, alias, isAlias = alias, intropDefault } = {}) {
requireModule(path, { esm, useESM = esm, alias, isAlias = alias, intropDefault, interopDefault = intropDefault } = {}) {
let resolvedPath = path
let requiredModule

// TODO: Remove in Nuxt 3
if (intropDefault) {
consola.warn('Using intropDefault is deprecated and will be removed in Nuxt 3. Use `interopDefault` instead.')
}
if (alias) {
consola.warn('Using alias is deprecated and will be removed in Nuxt 3. Use `isAlias` instead.')
}
Expand Down Expand Up @@ -148,8 +151,8 @@ export default class Resolver {
lastError = e
}

// Introp default
if (intropDefault !== false && requiredModule && requiredModule.default) {
// Interop default
if (interopDefault !== false && requiredModule && requiredModule.default) {
requiredModule = requiredModule.default
}

Expand Down
4 changes: 2 additions & 2 deletions packages/core/test/resolver.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -384,14 +384,14 @@ describe('core: resolver', () => {
expect(resolvedModule).toEqual('resolved module')
})

test('should require es modules without default export when intropDefault is disabled', () => {
test('should require es modules without default export when interopDefault is disabled', () => {
const resolver = new Resolver({
options: {}
})
resolver.resolvePath = jest.fn()
resolver.esm = jest.fn(() => ({ default: 'resolved module' }))

const resolvedModule = resolver.requireModule('/var/nuxt/resolver/module', { intropDefault: false })
const resolvedModule = resolver.requireModule('/var/nuxt/resolver/module', { interopDefault: false })

expect(resolvedModule).toEqual({ default: 'resolved module' })
})
Expand Down

0 comments on commit 569b6aa

Please sign in to comment.