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

menambahkan materi database - mysql, namun materi nya masih belum len… #40

Merged
merged 10 commits into from
Oct 28, 2021

Conversation

nersus15
Copy link
Contributor

@nersus15 nersus15 commented Oct 4, 2021

…gkap

  • saya menambahkan materi basic baru, materi database mysql

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 (im used):

  • os = linux

Testing

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

Maintainer

@nomadkode nomadkode requested review from a team October 4, 2021 03:37
@nomadkode nomadkode added basic enhancement New feature or request labels Oct 4, 2021
@wildanie12
Copy link
Member

Mention Issue #36

Iya, sementara kita tahan dulu PR nya ya ..
Kita akan buatkan silabus untuk materi-materi selanjutnya, setelah itu kita akan publish juga sebagai Issue yang nantinya bisa di assign secara public dan kita akan mention Issue ini juga sebagai sub-materi.

@nomadkode
Copy link
Contributor

di close aja kah PR nya? @wildanie12

@wildanie12
Copy link
Member

wildanie12 commented Oct 13, 2021

di close aja kah PR nya? @wildanie12

Jangan dulu om, PR ini bisa digunakan untuk sub materi pertama, hanya saja kita perlu menyiapkan format database yang terstruktur aja .. PR ini akan direview setelah Issue Relative link selesai.

@nersus15 nersus15 requested a review from wildanie12 as a code owner October 15, 2021 05:12
Copy link
Contributor

@mzaini30 mzaini30 left a comment

Choose a reason for hiding this comment

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

Sedikit perbaikan tipo.

basics/8_databases/mysql/README.md Outdated Show resolved Hide resolved
basics/8_databases/mysql/README.md Outdated Show resolved Hide resolved
basics/8_databases/mysql/README.md Outdated Show resolved Hide resolved
basics/8_databases/mysql/README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@msazzuhair msazzuhair left a comment

Choose a reason for hiding this comment

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

Mohon untuk mengikuti pedoman kontribusi terbaru.

Tidak perlu lagi menambahkan berkas index.php di setiap folder sejak pr #55

@wildanie12 wildanie12 linked an issue Oct 25, 2021 that may be closed by this pull request
4 tasks
@wildanie12
Copy link
Member

wildanie12 commented Oct 25, 2021

Saya mohon maaf bang @nersus15 karena terlalu lama Pull Request ini di biarkan dengan Justifikasi yang minimum, itu karena kami perlu menerapkan konsep dan ketentuan agar penggunaan database di repo Bellshade PHP ini bisa rapi, terstruktur dan scalable
dan beberapa ketentuan sudah kami tuliskan di pedoman berkontribusi.
Lihat selengkapnya -> Penggunaan database - CONTRIBUTING.md

Selanjutnya PR ini bisa tetep dibuka, dan saya yang akan melanjutkan hasil pekerjaan bang @nersus15 agar sesuai dengan Silabus yang kami tulis di Issue #67 atau lebih spesifiknya #36

@wildanie12 wildanie12 self-assigned this Oct 25, 2021
@hendraaagil hendraaagil marked this pull request as draft October 26, 2021 11:26
@wildanie12
Copy link
Member

Mohon maaf, karena PR ini sudah ada terlebih dahulu sebelum workflow CommitCheck maka menurut saya tidak ada pilihan selain membuat pengecualian khusus untuk PR ini. Jika dilakukan rebase, sejarah commit akan ditulis kembali yang mengakibatkan commit terdahulu kembali muncul disini karena adanya merge dari main sebelumnya.

@wildanie12 wildanie12 marked this pull request as ready for review October 28, 2021 09:41
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.

Sudah cukup baik dengan bantuan perbaikan yang saya lakukan

@wildanie12 wildanie12 merged commit 64b58dc into bellshade:main Oct 28, 2021
@wildanie12 wildanie12 linked an issue Oct 28, 2021 that may be closed by this pull request
2 tasks
@wildanie12
Copy link
Member

closes #97

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
basic enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2. koneksi database 1. Pengenalan - Database dasar prosedural
6 participants