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

docs: menambah dokumentasi baru soal CSS Icon #255

Closed
wants to merge 10 commits into from
Closed

docs: menambah dokumentasi baru soal CSS Icon #255

wants to merge 10 commits into from

Conversation

pamela-sarnia
Copy link
Collaborator

@pamela-sarnia pamela-sarnia commented Oct 3, 2022

Deskripsi (Description)

Checklist:

Umum:
  • Saya menambah kode basic terbaru.
  • Saya memperbaiki kode basic yang sudah ada.
  • Saya menambah template terbaru.
  • Saya memperbaiki template yang sudah ada.
  • Saya memperbaiki dokumentasi.
  • Saya menambah dokumentasi.
Contributor Requirements (Syarat Kontributor) dan Lain-Lain:
  • Saya sudah membaca (I have read) CONTRIBUTING dan sudah menyetujui semua syarat.
  • Saya telah menambahkan komentar kode yang memberikan penjelasan maksud dari kode yang saya buat.
  • Saya menggunakan bahasa Indonesia untuk memberikan penjelasan dari kode yang saya buat.

Environment

Saya menggunakan (I'm using):

  • os = windows

Testing

  • link html
  • lint css

linked issue #NOMOR_ISSUE

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
docs: menambah dokumentasi baru soal CSS Icon
@pamela-sarnia pamela-sarnia requested a review from dhafitf as a code owner October 3, 2022 18:08
@auto-add-label auto-add-label bot added the documentation Improvements or additions to documentation label Oct 3, 2022
Copy link
Member

@slowy07 slowy07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, tetapi salah dalam penempatan file README.md nya, jika di merge akan mengganti bagian README ini, file ini seharusnya dimasukkan dalam direktori CSS

untuk penambahannya silahkan dibuat pada direktori CSS dengan cara:

  • masuk ke direktori CSS
  • kemudian buat folder yang bernama 058 CSS google font
  • di dalam folder tersebut isi README.md yang berisi materi penjelasan tentang CSS google font

@pamela-sarnia
Copy link
Collaborator Author

LGTM, tetapi salah dalam penempatan file README.md nya, jika di merge akan mengganti bagian README ini, file ini seharusnya dimasukkan dalam direktori CSS

untuk penambahannya silahkan dibuat pada direktori CSS dengan cara:

  • masuk ke direktori CSS
  • kemudian buat folder yang bernama 058 CSS google font
  • di dalam folder tersebut isi README.md yang berisi materi penjelasan tentang CSS google font

oke sensei

@dhafitf
Copy link
Member

dhafitf commented Oct 4, 2022

LGTM, tetapi salah dalam penempatan file README.md nya, jika di merge akan mengganti bagian README ini, file ini seharusnya dimasukkan dalam direktori CSS

untuk penambahannya silahkan dibuat pada direktori CSS dengan cara:

  • masuk ke direktori CSS
  • kemudian buat folder yang bernama 058 CSS google font
  • di dalam folder tersebut isi README.md yang berisi materi penjelasan tentang CSS google font

Benar sekali, file README.md harus berada dalam folder CSS dan 021 CSS Icons, seperti yang sudah tercantum pada #204

@pamela-sarnia
Copy link
Collaborator Author

LGTM, tetapi salah dalam penempatan file README.md nya, jika di merge akan mengganti bagian README ini, file ini seharusnya dimasukkan dalam direktori CSS
untuk penambahannya silahkan dibuat pada direktori CSS dengan cara:

  • masuk ke direktori CSS
  • kemudian buat folder yang bernama 058 CSS google font
  • di dalam folder tersebut isi README.md yang berisi materi penjelasan tentang CSS google font

Benar sekali, file README.md harus berada dalam folder CSS dan 021 CSS Icons, seperti yang sudah tercantum pada #204

sudah pull request yang baru

@dhafitf
Copy link
Member

dhafitf commented Oct 4, 2022

LGTM, tetapi salah dalam penempatan file README.md nya, jika di merge akan mengganti bagian README ini, file ini seharusnya dimasukkan dalam direktori CSS
untuk penambahannya silahkan dibuat pada direktori CSS dengan cara:

  • masuk ke direktori CSS
  • kemudian buat folder yang bernama 058 CSS google font
  • di dalam folder tersebut isi README.md yang berisi materi penjelasan tentang CSS google font

Benar sekali, file README.md harus berada dalam folder CSS dan 021 CSS Icons, seperti yang sudah tercantum pada #204

sudah pull request yang baru

Baik, kalau begitu PR ini saya tutup.

@dhafitf dhafitf closed this Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation markdown files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants