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.

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

docs(id): Penerjemahan pada berkas guide/migration - Part 2 #253

Closed

Conversation

Elmerf
Copy link

@Elmerf Elmerf commented Aug 24, 2021

Lanjutan penerjemahaan pada issue #23 dan PR #251

Berkas yang diterjemahkan:

  • keycodes-modifiers
  • listeners-removed
  • props-default-this

@Elmerf Elmerf marked this pull request as ready for review August 28, 2021 02:30
@mandaputtra
Copy link
Member

Untuk file yarn.lock jangan di commit yak. Boleh minta tolong perubahan yg ada di yarn.lock di kembalikan aja? Bisa mengikuti ini https://stackoverflow.com/questions/215718/how-can-i-reset-or-revert-a-file-to-a-specific-revision.

Kenapa yarn.lock tidak boleh dirubah? Ini bakal banyak membuat conflict dengan upstream repo (vuejs/docs-next) yang nantinya bisa menyebabkan website tidak berhasil di build.

@Elmerf
Copy link
Author

Elmerf commented Aug 30, 2021

Oh begitu ya, maaf saya kurang tahu pentingnya yarn.lock jadi saya tidak tambahkan ke .gitignore. Saya coba revisi ulang terlebih dahulu. Terima kasih atas penjelasannya

@mandaputtra
Copy link
Member

Jangan ditambahkan ke .gitignore yak. Di revert aja.

@Elmerf
Copy link
Author

Elmerf commented Aug 30, 2021

Saya sedikit kebingungann, mohon bantuannya. Di sini saya sudah revert yarn.lock nya jadi sebelumnya. Terbukti di files changed beberapa versi 4.6.0 kembali ke versi 4.4.0, Nah masalahnya, berkas yang saya terjemahkan sebelumnya kembali ke bahasa Inggris lagi. Untuk mencegah ini bagaimana ya?

@Elmerf
Copy link
Author

Elmerf commented Sep 2, 2021

PR ini saya close saja, saya buatkan PR baru biar lebih tertata.

@Elmerf Elmerf closed this Sep 2, 2021
@Elmerf Elmerf deleted the elmerf/id-migration-guide-part-2 branch September 2, 2021 10:46
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.

2 participants