Skip to content

add: c profile-ARN182-23160 - #35

Open
marifzein wants to merge 1 commit into
hsiabdullahroy:mainfrom
marifzein:profile-ARN182-23160
Open

add: c profile-ARN182-23160#35
marifzein wants to merge 1 commit into
hsiabdullahroy:mainfrom
marifzein:profile-ARN182-23160

Conversation

@marifzein

Copy link
Copy Markdown
Contributor

menambahkan profile ARN182-23160

Closes #20

@fitrifityanto

Copy link
Copy Markdown
Contributor

bismillah kak @marifzein ,

afwan, ada beberapa hal yang perlu diluruskan ya:

Sebelumnya, kaka sudah pernah membuat PR untuk menambahkan data profil berdasarkan issue yang kaka buat sendiri, dan PR tersebut sudah di-merge.
Namun, saat itu kaka lupa/terlewat belum menambahkan Closes #20 di deskripsi PR, jadi issue-nya tidak tertutup secara otomatis.

Sekarang kaka membuat PR baru dari branch yang sesuai (ini sudah benar), dan sudah menambahkan Closes #20 di deskripsi. Tapi isi dari file profile-submission.json tidak sesuai dengan deskripsi PR-mu, karena data profil kaka sudah ditambahkan di PR sebelumnya. Jadi PR ini sebenarnya tidak menambahkan data baru (dan terimakasih sudah merapikan dengan memberikan tanda spasi disalah satu data profil lain)

Agar sesuai dengan petunjuk tugas (yaitu PR harus menutup issue yang dibuat), kaka bisa melakukan langkah berikut:

  • Ubah posisi data profil kaka di file profile-submission.json (misalnya pindahkan ke bagian bawah, agar file tetap berubah).

  • Mohon jangan menghapus atau menimpa data profil lain saat mengedit file.

  • Ubah deskripsi PR ini menjadi: Update data profil [NIP]

Setelah itu, PR ini akan valid untuk menutup issue secara otomatis lewat Closes #20.

Silakan diperbaiki ya. Semangat belajar Git

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.

Github Profile - [ARN182-23160]

2 participants