Skip to content

Conversation

@Ateng-labrador
Copy link
Contributor

Deskripsi (Description)

Saya menambahkan algoritma baru pada dictionary other

Checklist:

Umum:

  • Saya menambah algoritma terbaru.
  • Saya memperbaiki algoritma yang sudah ada.
  • Saya memperbaiki dokumentasi.
  • Saya menambah dokumentasi.

Contributor Requirements (Syarat Kontributor) dan Lain-Lain:

  • Saya sudah membaca (I have read) CONTRIBUTING dan sudah menyetujui semua syarat.
  • 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.

Unit Testing dan Linting:

  • pytest
  • flake8

Environment

Saya menggunakan (I'm using):

  • os = windows
  • python = py -3 -V (Windows-Python launcher)

linked issue #

Comment on lines 26 to 35
temp = number
total = 0
while temp > 0:
digit = temp % 10
total += factorial(digit)
temp //= 10
if total == number:
return benar
else :
return bukan
Copy link
Member

Choose a reason for hiding this comment

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

bisa pakai model list comprehension

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Jadi susah di pahamin nanti bang menurut ku

@mergify mergify bot requested a review from a team October 24, 2025 14:17
@Ateng-labrador
Copy link
Contributor Author

@slowy07 bisa di cek dulu bang,tapi kalo aku lebih suka yang bukan list compresion karna lebih enak di baca,rinci dan mudah di pahami 🙏 🙏 🙏

@mergify mergify bot requested a review from a team October 24, 2025 15:17
Co-authored-by: arfy slowy <slowy.arfy@proton.me>
@Ateng-labrador
Copy link
Contributor Author

@slowy07

Copy link
Member

@slowy07 slowy07 left a comment

Choose a reason for hiding this comment

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

LGTM, terima kasih atas kontribusinya @Ateng-labrador

@slowy07 slowy07 added the ready-to-merge ok to merge to branch main label Nov 13, 2025
@mergify mergify bot merged commit 000b42a into bellshade:main Nov 13, 2025
6 of 9 checks passed
@mergify
Copy link

mergify bot commented Nov 13, 2025

terima kasih atas kontribusinya @Ateng-labrador !

@mergify mergify bot requested a review from a team November 13, 2025 03:51
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.

2 participants