Skip to content

Add support for eslint-plugin-typescript#68

Merged
lydell merged 17 commits intomasterfrom
eslint-plugin-typescript
Jan 12, 2019
Merged

Add support for eslint-plugin-typescript#68
lydell merged 17 commits intomasterfrom
eslint-plugin-typescript

Conversation

@j-f1
Copy link
Copy Markdown
Member

@j-f1 j-f1 commented Jan 1, 2019

@lydell Can you take a look and see if I did anything wrong?

@j-f1 j-f1 requested a review from lydell January 1, 2019 00:45
Comment thread README.md Outdated
Comment thread README.md Outdated
Copy link
Copy Markdown
Member

@lydell lydell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! I usually order stuff alphabetically – would you mind doing that? Otherwise I'll do it after merging.

Comment thread test-lint/typescript.ts Outdated
Comment thread test/validators.test.js Outdated
@j-f1
Copy link
Copy Markdown
Member Author

j-f1 commented Jan 3, 2019

Removed 🔥

Comment thread .eslintrc.base.js Outdated
Comment thread .eslintrc.base.js
Comment thread .eslintrc.base.js
@j-f1 j-f1 force-pushed the eslint-plugin-typescript branch from 86923ec to 50dcfc8 Compare January 11, 2019 20:48
@lydell lydell merged commit c77f52e into master Jan 12, 2019
@lydell lydell deleted the eslint-plugin-typescript branch January 12, 2019 17:47
@IlyaSemenov
Copy link
Copy Markdown

Can this be published to npm now? Thanks. (By the way did you consider https://github.com/semantic-release/semantic-release ?)

@lydell
Copy link
Copy Markdown
Member

lydell commented Jan 13, 2019

I'll make a release later today.

@lydell
Copy link
Copy Markdown
Member

lydell commented Jan 13, 2019

Release in v3.4.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants