Nicholas C. Zakas nzakas

Organizations

@eslint @todogroup
nzakas commented on issue eslint/eslint#2195
@nzakas

Thanks, we should get that updated.

nzakas commented on pull request eslint/eslint.github.io#49
@nzakas

Do you want to make that change plus the change @ilyavolodin mentioned?

nzakas commented on pull request eslint/eslint#2175
@nzakas

Good points. We've run into trouble in the past trying to use AST nodes for things like this, that's my only concern. I keep going back and forth …

nzakas commented on issue eslint/eslint#2158
@nzakas

You're saying that current ES6 implementations have bugs in this regard. I'm saying ES6 specifies this behavior so it will eventually be correct ev…

nzakas commented on pull request eslint/eslint#2159
@nzakas

Of that's true, please file an issue with escope. I really don't want to be doing things like this.

nzakas commented on pull request eslint/eslint#2159
@nzakas

This skips every variable that was used? Why?

nzakas commented on pull request eslint/eslint#2159
@nzakas

This section needs to show an example and mention const. The word "constant" is too ambiguous to stand alone.

nzakas commented on pull request eslint/eslint#2159
@nzakas

Disallows reassignment of const declarations.

nzakas commented on pull request eslint/eslint#2159
@nzakas

Still think this should be no-const-reassign as it relates only to const (and also, distinguishes it from no-constant-condition, which has nothing …

nzakas commented on issue eslint/eslint#2187
@nzakas

necessitates a comment in that block? Yes. would /**/ work, or does the comment have to have content? With #2193 that comment will work.

nzakas commented on issue eslint/eslint#2194
@nzakas

Imports/exports have so many variations, it's crazy. :)

nzakas pushed to master at eslint/eslint
@nzakas
nzakas closed issue eslint/eslint#2190
@nzakas
proprty typo in camelcase.md
nzakas merged pull request eslint/eslint#2192
@nzakas
Docs: Fix spell in camelcase doc (fixes #2190)
1 commit with 1 addition and 1 deletion
nzakas commented on pull request eslint/eslint#2192
@nzakas

Thanks!

nzakas pushed to master at eslint/eslint
@nzakas
nzakas closed issue eslint/eslint#2188
@nzakas
`no-empty` rule should allow empty blocks with any single comment
nzakas merged pull request eslint/eslint#2193
@nzakas
Fix: `no-empty` rule should allow any comments (fixes #2188)
1 commit with 11 additions and 10 deletions
nzakas commented on pull request eslint/eslint#2193
@nzakas

:+1: thanks @gyandeeps

nzakas commented on issue karma-runner/karma#1078
@nzakas

Yeah, there's definitely a trend towards not doing global installs for CLIs because it makes projects no longer self-contained.

nzakas commented on pull request eslint/eslint.github.io#49
@nzakas

Can you explain what looks odd?

nzakas commented on issue nzakas/understandinges6#153
@nzakas

In this case mixin() is referring to the function of the same name in the immediate preceding example.

nzakas closed issue eslint/espree#116
@nzakas
Destructuring and spread operator: Unexpected token ..
nzakas commented on issue eslint/espree#116
@nzakas

We only support JSX as defined in the spec: http://facebook.github.io/jsx/ The spread object operator isn't part of ES6, and isn't part of the JSX …

nzakas pushed to master at eslint/eslint
@nzakas
nzakas closed issue eslint/eslint#2174
@nzakas
Require semicolon after import statements
nzakas merged pull request eslint/eslint#2178
@nzakas
Fix: Require semicolon after import/export statements (fixes #2174)
1 commit with 22 additions and 3 deletions
nzakas commented on pull request eslint/eslint#2178
@nzakas

LGTM. Thanks!

nzakas commented on issue eslint/eslint#2191
@nzakas

Can you please paste the console output as well? It helps us to see what's actually being flagged.

nzakas commented on issue eslint/eslint#1841
@nzakas

Just to address our concerns: I'm sorry you feel like the linter is forcing you to make changes you don't want. We do our best to assure that our r…