Skip to content

Tsconfig zh #1268

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 8 commits into from
Oct 29, 2020
Merged

Tsconfig zh #1268

merged 8 commits into from
Oct 29, 2020

Conversation

Kingwl
Copy link
Contributor

@Kingwl Kingwl commented Oct 22, 2020

Add

  • copy for zh
  • allowSyntheticDefaultImports
  • esModuleInterop

They are very confusing options. So I have translated them first.

@github-actions
Copy link
Contributor

Thanks for the PR!

This section of the codebase is owned by @Kingwl - if they write a comment saying "LGTM" then it will be merged.

@orta
Copy link
Contributor

orta commented Oct 22, 2020

Good luck, you can decide yourself when you want to merge 👍🏻

@Kingwl
Copy link
Contributor Author

Kingwl commented Oct 28, 2020

Is everything ok?
I'd like to merge this if no more append after 24:00 UTC+8.

@orta
Copy link
Contributor

orta commented Oct 28, 2020

Sounds cool to me - you can tell the bot to merge when you feel good 👍🏻

@orta
Copy link
Contributor

orta commented Oct 28, 2020

( That said, my recommendation is to delete the English ones - then you know you have the latest versions when you translate them ( #1275 for example just added and amended a bunch ) and #296 will be accurate on the TODO )

@Kingwl
Copy link
Contributor Author

Kingwl commented Oct 29, 2020

There's no new changes(only remove other files). And no more append comment.
So (I'm not sure it's work).

LGTM.

@github-actions github-actions bot merged commit b2a5dcf into microsoft:v2 Oct 29, 2020
@github-actions
Copy link
Contributor

Merging because @Kingwl is a code-owner of all the changes - thanks!

@Kingwl Kingwl deleted the tsconfig-zh branch October 29, 2020 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants