Skip to content

chore: add lint-ts rule#10427

Merged
nicolo-ribaudo merged 1 commit intobabel:masterfrom
JLHwung:add-lint-ts
Sep 11, 2019
Merged

chore: add lint-ts rule#10427
nicolo-ribaudo merged 1 commit intobabel:masterfrom
JLHwung:add-lint-ts

Conversation

@JLHwung
Copy link
Copy Markdown
Contributor

@JLHwung JLHwung commented Sep 10, 2019

Q                       A
Tests Added + Pass? Yes
License MIT

Add lint-ts to lint rule. Basically it runs typescript compiler on generated babel types index.d.ts and reports any errors. This should prevent #10403 happens again.

@JLHwung JLHwung added the PR: Internal 🏠 A type of pull request used for our changelog categories label Sep 10, 2019
@nicolo-ribaudo
Copy link
Copy Markdown
Member

Generated flow types are already checked by make flow, right?

@babel-bot
Copy link
Copy Markdown
Collaborator

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/11544/

@JLHwung
Copy link
Copy Markdown
Contributor Author

JLHwung commented Sep 10, 2019

Generated flow types are already checked by make flow, right?

Yes. make flow will check the flow types in the project scope. But typescript files are only limited to babel-types and it is auto-generated, so I don't think it suffices to add make typescript unless we have typescript in our codebase.

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

Labels

outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: Internal 🏠 A type of pull request used for our changelog categories

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants