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

Tambahkan antarmuka CLI #7

Closed
wants to merge 0 commits into from
Closed

Conversation

prasastoadi
Copy link
Contributor

Menambahkan antarmuka CLI #6

Penggunaan melalui CLI.

$ python kbbi.py --kata=cinta

Pencarian frasa yang dipisahkan oleh spasi harus diapit oleh tanda petik.

$ python kbbi.py --kata="tanggung jawab"

Menambahkan contoh kalimat.

$ python kbbi.py --kata="tanggung jawab" --contoh

@laymonage
Copy link
Owner

Hai @prasastoadi, terima kasih atas pull request-nya! 👍

Saya belum sempat mencoba, tetapi ada beberapa hal yang ingin saya tanyakan:

  • Apakah abseil bisa digantikan dengan pustaka bawaan Python, misalnya argparse? Saya rasa akan lebih baik apabila kita tidak menambah dependensi dari luar, kecuali memang benar-benar diperlukan. Sebagian besar pengguna modul ini memanfaatkannya sebagai semacam API untuk KBBI, jadi saya tidak ingin memberatkan mereka dengan menambah dependensi yang tidak mereka perlukan.
  • Apakah pencarian bisa dilakukan tanpa menuliskan --kata= secara eksplisit? Jadi cukup python kbbi.py "tanggung jawab"
  • Bagaimana dengan --json? 😉

Poin bonus kalau bisa menambahkan executable script supaya modul bisa langsung dipanggil (kbbi, bukan python kbbi.py) 😄

@laymonage
Copy link
Owner

@prasastoadi, saya izin melanjutkan dan force push ke branch ini ya. 🙏

@laymonage
Copy link
Owner

Wah, saya salah push jadi PR-nya otomatis tertutup dan saya kehilangan akses ke branch ini. Saya merge langsung saja ya. Mohon maaf 😅

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