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

delete folder vendor in git #18

Closed
wants to merge 3 commits into from

Conversation

ralfiannor
Copy link

@ralfiannor ralfiannor commented Aug 8, 2019

Sepengetahuan saya folder vendor adalah hasil build ketika kita menjalankan perintah "composer install". Nah perintah itu harusnya dilakukan di local komputer masing-masing sehingga apa yang kita download dan upload di git hanya source code yang berisi core nya saja.

Untuk masalah https://github.com/OpenSID/OpenDK/wiki/Install-di-cPanel sebaiknya kita buat halaman release dimana disana berisi file zip aplikasi yang sudah rilis bukan download source code untuk developer. Seperti pada OpenSID https://github.com/OpenSID/OpenSID/releases

@agungsugiarto
Copy link
Member

Pak @eddieridwan mohon untuk di merge pr ini.

@eddieridwan
Copy link
Contributor

@ralfiannor , maaf baru sempat kembali memperhatikan OpenDK. Terima kasih PR-nya. Tapi setelah saya coba, banyak yg tidak jalan, karena ada yg kurang setelah folder vendor dihapus. Mohon bantuannya untuk mencoba memperbaiki PR-nya supaya bisa jalan setelah folder vendor dihapus.

@eddieridwan
Copy link
Contributor

Bung @ralfiannor, apakah anda sempat membantu memperbaiki OpenDK? Kalau sempat, saya berikan anda akses bisa push langsung ke repo OpenDK, untuk mempercepat.

@eddieridwan
Copy link
Contributor

@ralfiannor , penghapusan folder vendor telah diterapkan melalui #35, sekalian naikkan versi Laravel.

Karena itu, pull request ini akan ditutup. Tapi silakan buka lagi kalau ada yg perlu ditambah.

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.

3 participants