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

feat: Menambahkan materi array manipulation #157

Merged
merged 4 commits into from
Oct 21, 2021
Merged

feat: Menambahkan materi array manipulation #157

merged 4 commits into from
Oct 21, 2021

Conversation

grafisaholic
Copy link
Contributor

@grafisaholic grafisaholic commented Oct 20, 2021

Closes

Deskripsi (Description)

  • saya menambahkan algoritma terbaru
  • saya memperbaiki algoritma yang sudah ada
  • saya menambahkan materi/sub materi basic atau ES6
  • saya memperbaiki materi/sub materi basic atau ES6
  • saya menambah dokumentasi
  • saya memperbaiki dokumentasi

Contributor Requirements (Syarat Kontributor)

  • saya sudah membaca (i have read) CONTRIBUTING dan sudah menyetujui semua
  • saya telah menambahkan docstring yang memberikan penjelasan maksud dari kode yang saya buat
  • saya menggunakan bahasa indonesia untuk memberikan penjelasan dari kode yang saya buat

Environment

saya menggunakan (im used):

  • os = linux

Testing

  • doctesting js
  • lint testing eslint

Maintainer

@bellshade/javascript-team
@bellshade/docs-team

@auto-add-label auto-add-label bot added the enhancement New feature or request label Oct 20, 2021
@nomadkode nomadkode requested review from a team October 20, 2021 16:09
Copy link
Member

@reacto11mecha reacto11mecha left a comment

Choose a reason for hiding this comment

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

Terimakasih sudah berkontribusi ke bellshade repo js 👍, untuk kode js kiranya sudah aman. Untuk docs ada typo sedikit, bisa dibenerin dulu.

learn/basic/010_array_manipulation/sort.js Outdated Show resolved Hide resolved
Copy link
Collaborator

@rizkyikhwan rizkyikhwan left a comment

Choose a reason for hiding this comment

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

Di penjelasannya terdapat kalimat atau kata yang menurut saya kurang tepat dan efektif. Untuk kodenya sendiri tidak ada masalah menurut saya.


/**
* PENJELASAN :
* SORT dalam bahasa indonesia berarty mengurutkan
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* SORT dalam bahasa indonesia berarty mengurutkan
* SORT dalam bahasa indonesia artinya mengurutkan

Menurut saya menggunakan kata “berarti” kurang tepat untuk mengartikan sebuah kata bahasa inggris/asing.

/**
* PENJELASAN :
* SORT dalam bahasa indonesia berarty mengurutkan
* sortAscending bertujuan untuk mengurutkan angka dalam array menjadi urutan dari yang terkecil ke terbesar
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* sortAscending bertujuan untuk mengurutkan angka dalam array menjadi urutan dari yang terkecil ke terbesar
* sortAscending bertujuan untuk mengurutkan angka dalam array dari yang terkecil hingga terbesar.

* PENJELASAN :
* SORT dalam bahasa indonesia berarty mengurutkan
* sortAscending bertujuan untuk mengurutkan angka dalam array menjadi urutan dari yang terkecil ke terbesar
* sortDescending bertujuan untuk mengurutkan angka dalam array menjadi urutan dari yang terbesar ke terkecil
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* sortDescending bertujuan untuk mengurutkan angka dalam array menjadi urutan dari yang terbesar ke terkecil
* sortDescending bertujuan untuk mengurutkan angka dalam array dari yang terbesar hingga terkecil.

Merubah menjadi kalimat yang efektif

Copy link
Contributor Author

Choose a reason for hiding this comment

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

maaf merepotkan

@reacto11mecha
Copy link
Member

Pesan commitnya diperhatikan, ganti dulu pesan commitnya sesuai standar yang udah ada ya

git commit --amend -m "fix: fixing typo docs description"
git push -f

Baca juga #58

@grafisaholic grafisaholic requested review from reacto11mecha and removed request for a team October 20, 2021 19:13
@reacto11mecha
Copy link
Member

Buat benerin pesan commit yang headnya 2671e99 itu perlu dilakuin rebase interactive.

git rebase --interactive '2671e99^'

Nanti bakalan ada tulisan

pick 2671e99

Ubah tulisan pick jadi edit, abis itu save. Setelahnya baru bisa di amend dan rebase continhe.

git commit --amend -m "fix: fixing typo docs description"
git rebase --continue

Abis itu push force deh

git push -f

@reacto11mecha
Copy link
Member

Tolong diubah ya, kalo ga ada perubahan, seminggu dari hari ini PR nya akan di close.

Terimakasih

@grafisaholic
Copy link
Contributor Author

terimakasih atas panduannya, maaf jika sering salah karena masih awam 🙏🏻

Copy link
Member

@reacto11mecha reacto11mecha left a comment

Choose a reason for hiding this comment

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

Sip, LGTM 👍

Copy link
Contributor

@nomadkode nomadkode left a comment

Choose a reason for hiding this comment

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

Penjelasannya boleh ditambahin lagi yaa, bisa melihat dari materi2 di folder lain sebagai acuan penjelasan.

Copy link
Member

@reacto11mecha reacto11mecha left a comment

Choose a reason for hiding this comment

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

Much better, terimakasih atas kontribusinya 👍

@slowy07 slowy07 merged commit 9c2d812 into bellshade:main Oct 21, 2021
@grafisaholic grafisaholic deleted the dev branch October 21, 2021 10:53
@grafisaholic grafisaholic restored the dev branch October 21, 2021 10:53
@grafisaholic grafisaholic deleted the dev branch August 4, 2022 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants