WIP: Bump versions of babel-eslint, eslint and eslint-config-canonical #359
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I bumped versions of
babel-eslint
,eslint
andeslint-config-canonical
. Becausebabel-eslint
have changed their scope analysis in this commit to visit also some type annotations, some tests were failing (related issue: #336), in particular,define-flow-type
anduse-flow-type
.Tests in
use-flow-type
rule were failing because theno-unused
lint error wasn't being thrown anymore. Tests indefine-flow-type
were failing because theno-undef
lint error wasn't being thrown anymore. I removed the tests.So these changes were required to get our tests working with new parser and eslint. However, I think that with a pull request to
babel-eslint
we will be able to get rid ofdefine-flow-type
anduse-flow-type
rules entirely. I filed the issue few months ago without a response so I will try to create a pull request with needed changes. If they get approved, then we can remove mentioned rules and publish new major version of the plugin. I will update this pull request accordingly.Btw, there is already
v10.0.0
version ofbabel-eslint
but I couldn't use it because this commit started to renameTypeAlias
nodes intoFunctionDeclaration
and it of course broke a lot of our rules.Because of
eslint-config-canonical
update, I had to apply lint fixes to a lot of files. It kind of messes this pull request, however, I wasn't able to separate lint and bump changes into two commits. the most important files of this pull request are source and test files fordefine-flow-type
anduse-flow-type
.