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

Tambah penerangan tentang environment variables ketika pemasangan pelayan. #12

Merged
merged 2 commits into from
Sep 23, 2022
Merged

Tambah penerangan tentang environment variables ketika pemasangan pelayan. #12

merged 2 commits into from
Sep 23, 2022

Conversation

afiqzx
Copy link

@afiqzx afiqzx commented Aug 31, 2022

Pelayan ketika ini sebenarnya memerlukan environment variables untuk berfungsi dengan betul. Akan tetapi, tiada penerangan tentang menetapkan environment variables di README.md.

Untuk siapa perubahan ini?

Perubahan ini untuk memudahkan penyumbang baharu yang tidak berapa memahami python, poetry atau sistem operasi secara tetap.

Adakah perubahan ini menyelesaikan masalah?

Secara jujurnya, tidak. Seperti dinyatakan di atas, ini hanyalah untuk memudahkan penyumbang baharu sementara fungsi lain yang lebih penting diselesaikan terlebih dahulu. Rasanya menetapkan nilai lalai boleh menyelesaikan masalah ini, tetapi saya mahu fokus pada fungsi lain terlebih dahulu.

@Thaza-Kun
Copy link
Collaborator

Elok juga itu. Kalau ada apa-apa penambahan dokumentasi yang dirasakan perlu, tambah sahajalah di sini. Dalam masa satu atau dua minggu nanti kita merge kalau tiada penambahan.

@Thaza-Kun Thaza-Kun marked this pull request as draft September 2, 2022 07:09
@Thaza-Kun Thaza-Kun added the ⚙️ Dokumentasi Penambahbaikan dokumentasi kod label Sep 2, 2022
@Thaza-Kun Thaza-Kun linked an issue Sep 2, 2022 that may be closed by this pull request
@Thaza-Kun
Copy link
Collaborator

Sudah tambah penerangan tentang fail ./.env dan kepentingannya. Boleh @afiqzx cuba ikut nak tahu sama ada arahannya boleh difahami atau tidak.

@afiqzx
Copy link
Author

afiqzx commented Sep 13, 2022

Terima kasih. Saya jarang guna .env dalam project yang saya kendalikan, lupa pula. 😅

Saya akan cuba lihat semula hujung minggu ini.

@afiqzx
Copy link
Author

afiqzx commented Sep 22, 2022

Saya sudah pastikan yang file .env boleh digunakan. Terima kasih kerana betulkan dokumentasi.

@Thaza-Kun
Copy link
Collaborator

Baiklah. Terima kasih @afiqzx. Memandangkan tiada ralat, saya andaikan arahannya jelas.

@Thaza-Kun Thaza-Kun closed this Sep 23, 2022
@Thaza-Kun Thaza-Kun reopened this Sep 23, 2022
@Thaza-Kun Thaza-Kun marked this pull request as ready for review September 23, 2022 11:52
@Thaza-Kun Thaza-Kun merged commit 76b8d5c into samudradev:master Sep 23, 2022
Thaza-Kun added a commit that referenced this pull request Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️ Dokumentasi Penambahbaikan dokumentasi kod
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dokumentasi Ringkas
2 participants