Skip to content

Enforce import type for the type imports #2026

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jun 30, 2020

Conversation

BatDroid
Copy link
Contributor

Summary

related to this issue.

Test plan

@callstack-bot
Copy link

callstack-bot commented Jun 29, 2020

Hey @BatDroid, thank you for your pull request 🤗. The documentation from this branch can be viewed here.

@BatDroid BatDroid force-pushed the enforce-import-type branch from b9edfcd to 2d35b09 Compare June 29, 2020 22:12
package.json Outdated
@@ -67,8 +67,8 @@
"@typescript-eslint/parser": "^2.12.0",
"all-contributors-cli": "^6.1.1",
"babel-cli": "^6.26.0",
"babel-core": "^7.0.0-bridge.0",
"babel-jest": "^24.3.1",
"babel-core": "^6.26.3",
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is it downgraded?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

maybe it has happened after conflict resolve. I'll fix it now.

Copy link
Contributor

Choose a reason for hiding this comment

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

Awesome, thanks a lot 🙏

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pushed. And thank you for letting me doing it. Hope I'll be able to contribute more :)

@Trancever Trancever merged commit f6f79d4 into callstack:4.0 Jun 30, 2020
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