-
-
Notifications
You must be signed in to change notification settings - Fork 177
UI server, memperbarui script dan update hal kecil lainnya #171
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
Conversation
Yang mau nyoba branch ini, caranya kayak begini. # https
git remote add ezra https://github.com/reacto11mecha/Javascript.git
# ssh
git remote add ezra git@github.com:reacto11mecha/Javascript.git Baru track branchnya git fetch ezra
git checkout --track ezra/ui-server Kalo semisal aku buat perubahan, tinggal pull aja di branch yang git pull |
ditunggu dan akan di coba |
This pull request introduces 4 alerts when merging 717a8ba into d329ba7 - view on LGTM.com new alerts:
|
This pull request introduces 4 alerts when merging 0743fe6 into 2f4e170 - view on LGTM.com new alerts:
|
This pull request introduces 8 alerts when merging b3871bc into 2f4e170 - view on LGTM.com new alerts:
|
gg untuk kerja kerasnya om @reacto11mecha |
This pull request introduces 4 alerts when merging d153a81 into 8aa97dc - view on LGTM.com new alerts:
|
This pull request introduces 4 alerts when merging 6a5062a into 571f373 - view on LGTM.com new alerts:
|
This pull request introduces 4 alerts when merging db49cd9 into 814313a - view on LGTM.com new alerts:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sip kalau gitu kayak nya udah semua
yaml nya bagian label perlu di edit ada bot mergefy yg error karena sudah gak ada tim docs. |
Okey, sekalian sama yang ini |
Selesai om, silahkan di review |
Udah oke sih harusnya. Cuma masih error. Help @slowy07 |
Kalo buat rule aman sih harusnya, dia perlu di merge dulu biar bisa kepake. Yang masalah aku ga ngerti di bagian node ci, kenapa dia failed |
Yaudah merge aja zra, kalo masih error bisa di solve nanti. |
Approve dulu om, terus kasih label. Dah aman kan docs nya |
Waduh baru aja nemu yang salah om. |
Node CI nya failed jadi walaupun di kasih label |
ngebuildnya masih failed yaa brarti? |
Hooh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
baru ada waktu buat nyobain ini tadi, LGTM om
Jadi mau di merge dulu apa solve dulu nih masalah node CI nya? Kan bisa force merge pake manual kan. |
Merge aja dulu, bisa di solve nanti |
Closes #164
Deskripsi (Description)
Contributor Requirements (Syarat Kontributor)
Environment
saya menggunakan (im used):
os
=linux
Testing