Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[KO-korean] README translation (Chapter 1, 5) #157

Merged
merged 10 commits into from
Dec 15, 2021
Merged

[KO-korean] README translation (Chapter 1, 5) #157

merged 10 commits into from
Dec 15, 2021

Conversation

SoniaComp
Copy link
Contributor

@SoniaComp SoniaComp commented Oct 3, 2021 •
edited
Loading

[ My Three Translation Principles ]

  1. I translated the literal translation in principle.
  2. Change to a term widely used in the industry.
  3. I paraphrased only when its meaning is ambiguos.

[#197] Chapter 1, 5

  • Introduction base README
    • Defining Data Science README
    • Defining Data Science assignment
    • Ethics README
    • Ethics assignment
    • Defining Data README
    • Defining Data assignment
    • Stats and Probability README
    • Stats and Probability assignment
  • Working with Data base README (assigned by [KO-korean] 2. Working-With-Data translation #167)
  • Vizualization base README
  • Data Science Life cycle
  • Data Science in the Cloud base README
    • Intro README
    • Intro assignment
    • Low Code README
    • Low Code assignment
    • Azure README
    • Azure assignment
  • Data Science in the Wild base

@jlooper
Copy link
Contributor

jlooper commented Oct 3, 2021

Wonderful! Could you turn this into a draft? 🙏

@jlooper jlooper marked this pull request as draft October 4, 2021 00:27
@SoniaComp
Copy link
Contributor Author

Sure! Thank you for your comment 😊

@SoniaComp SoniaComp changed the title [WIP] [KO-korean] translation [WIP] [KO-korean] README translation Oct 7, 2021
@SoniaComp
Copy link
Contributor Author

@jlooper Could you add the ‘hacktoberfest’ label to this PR?
I will finish this PR within this month :-)

@jlooper
Copy link
Contributor

jlooper commented Oct 7, 2021

@SoniaComp - Jasmine added it in :) thanks!

@SoniaComp
Copy link
Contributor Author

@jlooper Thank you so much 😊

@SoniaComp SoniaComp changed the title [WIP] [KO-korean] README translation [WIP] [KO-korean] README translation (1,3) Oct 17, 2021
@SoniaComp SoniaComp changed the title [WIP] [KO-korean] README translation (1,3) [WIP] [KO-korean] README translation (Chapter 1, 5) Oct 17, 2021
This was referenced Nov 7, 2021
Open
Merged
@SoniaComp
Copy link
Contributor Author

SoniaComp commented Dec 7, 2021 •
edited
Loading

I apologize for late upload 😭. I think I will finish it until this Sunday.

@SoniaComp
Copy link
Contributor Author

SoniaComp commented Dec 12, 2021 •
edited
Loading

@jlooper I translated all assignments, and READMEs of Chapter 1 and 5.
Sorry for late upload and thank you for giving me chance to contribute to this project.

@SoniaComp SoniaComp marked this pull request as ready for review December 12, 2021 14:19
paladique
paladique requested changes Dec 14, 2021
View reviewed changes
Copy link
Member

@paladique paladique left a comment

Choose a reason for hiding this comment

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

Wow, thanks so much for your work on this!! Just a few small comments on this and it will be ready to merge once resolved.

5-Data-Science-In-Cloud/translations/README.ko.md Outdated Show resolved Hide resolved
1-Introduction/translations/README.ko.md Outdated Show resolved Hide resolved
@SoniaComp
Copy link
Contributor Author

@paladique Thanks a lot for your checking 🙏 I solved all of the problems, and you can resolve conversation 😊

@paladique paladique merged commit 6e32a3c into microsoft:main Dec 15, 2021
@paladique paladique changed the title [WIP] [KO-korean] README translation (Chapter 1, 5) [KO-korean] README translation (Chapter 1, 5) Dec 15, 2021
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@paladique paladique paladique approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SoniaComp @jlooper @paladique