Replace eslint-plugin-typescript with @typescript-eslint/eslint-plugin

[tani]

In a few days ago, ESLint launched new project typescript-eslint which uses the new namespace @typescript-eslint/rule-name for rules. This pull request is for that changes. Cheers!

[KagamiChan]

seems tests are failing because @typescript-eslint/eslint-plugin is not listed as dev dep

while we might check existence of @typescript-eslint/eslint-plugin and determine which rule set is used, a breaking change is better IMO

[lydell]

I think we can make a breaking change. I need to make a major version bump anyway for #71.

[tani]

@KagamiChan and @lydell
I see, I remove old rules and update dependencies.

[tani]

I change eslint-plugin-typescript to @typescript-eslint/eslint-plugin to pass test, but I don't change version number because it may be able to do it at #71.

[lydell]

Nice!

I don't change version number

Tip: Never change the version number in a PR to any project. That’s typically done in a separate commit before releasing.

[tani]

I finish my work to do for this pull request.

• document README.md
• test test/*.js
• dependencies package.json and package-lock.json

[JamesHenry]

@lydell are you happy with this PR now?

[lydell]

Thanks! I’ll try to make the v4 release this weekend.

[tani]

Thank you! I'm looking forward to applying this change to my project!

[lydell]

v4.0.0 is now released!

[tani]

Wow! Thank you! 2019年1月26日(土) 19:56 Simon Lydell <notifications@github.com>:

…

v4.0.0 is now released! — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#74 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEyY_kwQ9OgwI9MMYMRZcEgPNqceKrlTks5vHDRVgaJpZM4aJ_l4> .