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

fix: file-file md yang belum sesuai standar linter #203

Merged
merged 3 commits into from
Mar 5, 2024

Conversation

sensasi-delight
Copy link
Collaborator

Closes

Deskripsi (Description)

  • saya menambahkan materi basic baru
  • saya memperbaiki materi basic yang sudah ada
  • saya menambahkan materi utility baru
  • saya memperbaiki materi utility sudah ada
  • saya menambahkan dokumentasi/artikel baru
  • saya memperbaiki dokumentasi/artikel yang sudah ada
  • saya menambahkan algoritma baru
  • saya memperbaiki algoritma yang sudah ada

Contributor Requirements (Syarat Kontributor)

  • saya sudah membaca (I have read) CONTRIBUTING dan sudah menyetujui semua
  • saya telah menambahkan docstring yang memberikan penjelasan maksud dari kode yang saya buat
  • saya sudah membuat artikel README.md tentang materi yang saya buat
  • saya menggunakan bahasa indonesia untuk memberikan penjelasan dari kode yang saya buat

Environment

saya menggunakan (I am using):

  • os = Windows 10

Testing

  • Codesniffer PSR-12 phpcs
  • Codesniffer autofix phpcbf
  • Unit testing PHPUnit

Maintainer

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 untuk markdownnya @sensasi-delight , mergify mungkin issue di konfigurasinya, bisa sekalian juga diganti konfigurasi dari mergify.yml pada line 58 yang sebelumnya

- name: review docs team untuk file markdown
    conditions:
      - label=markdown files
    actions:
      request_reviews:
        users:
          - ikr4mm
          - azqilana

menjadi

- name: review docs team untuk file markdown
    conditions:
      - label=markdown files
    actions:
      request_reviews:
        teams:
            - "@bellshade/php-team"

kemungkinan ketika sudah di change, akan dilakukan squash secara manual tanpa menggunakan mergify

@sensasi-delight
Copy link
Collaborator Author

@slowy07 hanya php-team atau sekalian dengan docs-team ya sensei? saya masih anak baru banh 😭

@slowy07
Copy link
Member

slowy07 commented May 6, 2023

iyaa bener tinggal ganti ke user review ke php-team atau ke docs-team, kalo saran boleh aja tambah keduanya hanya salah satunya @sensasi-delight

@sensasi-delight
Copy link
Collaborator Author

@slowy07 konfigurasi bernama review docs team untuk file markdown telah saya nonaktifkan karena tidak ada @bellshade/docs-team. Sedangkan untuk php-team telah terdapat pada konfigurasi review php team untuk file markdown pada baris 76-82

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 💯 , tested mergify work on #207

Reviewed slowy07@googlebot.com

Copy link
Member

@wildanie12 wildanie12 left a comment

Choose a reason for hiding this comment

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

Huge sorry for the late update. LGTM 🌟🚀. on behalf of docs & php team

@wildanie12
Copy link
Member

hmm docs team udah ngga ada lagi, any idea biar pass checknya sens @slowy07 ?

@sensasi-delight
Copy link
Collaborator Author

hmm docs team udah ngga ada lagi, any idea biar pass checknya sens @slowy07 ?

sementara dinonaktifkan dengan penjelasan seperti berikut:

penonaktifan akan efektif setelah PR ini di-merge. atau bisa juga dibentuk segera @bellshade/docs-team untuk repo ini.

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, waiting for merge

@slowy07 slowy07 merged commit b26cca1 into bellshade:main Mar 5, 2024
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants