naskya/napi-rs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge pull request #455 from napi-rs/new-without-scope

Browse source 
fix(cli): new command without npm scope
This commit is contained in:
LongYinan 2021-02-06 14:24:54 +08:00 committed by GitHub
commit 5a5c51f29e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
cli/src/new/index.ts
View file

@ -71,7 +71,8 @@ export class NewProjectCommand extends Command {
async execute() {
await this.getName()
if (!this.dirname) {
const [, defaultProjectDir] = this.name?.split('/') ?? []
const [scope, name] = this.name?.split('/') ?? []
const defaultProjectDir = name ?? scope
const dirAnswer = await prompt({
type: 'input',
name: DIR_PROMOTE_NAME,
@ -118,7 +119,8 @@ export class NewProjectCommand extends Command {
mkdirSync(join(process.cwd(), this.dirname!, 'src'))
}
const [, binaryName] = this.name!.split('/')
const [s, pkgName] = this.name!.split('/')
const binaryName = pkgName ?? s
this.writeFile('Cargo.toml', createCargoContent(this.name!))
this.writeFile('.npmignore', NPMIgnoreFiles)