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
Rename app config #98
Conversation
80c2a77
to
6b23802
Compare
bin/eslint-github-init.js
Outdated
@@ -47,7 +47,7 @@ const questions = [ | |||
} | |||
] | |||
|
|||
inquirer.prompt(questions).then(answers => { | |||
for (const answers of inquirer.prompt(questions)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this need an await
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh shoot, this was supposed to be a await/async
conversion and not a for-of
one.
bin/github-lint.js
Outdated
process.stderr.write(`\nCommand failed: ${nonzero}\n`) | ||
process.exit(nonzero) | ||
} | ||
}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't look right. There needs to be parens to call this function
The description of the
app
config is:Even though we have a
browser
config already. This PR renames theapp
config, theinternal
config and makes sure that it only loads rules that internal github applications would want.Also removed the
es6
part of the docs because I missed them in #96