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.

Sign up for GitHub

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

Jump to bottom

add: penjelasan deklarasi slices dengan sintaksis low:high #35

Merged
merged 1 commit into from Oct 14, 2023
Merged

add: penjelasan deklarasi slices dengan sintaksis low:high #35

merged 1 commit into from Oct 14, 2023

Conversation

malvinval
Copy link
Contributor

Deskripsi (Description)

Checklist:

Umum:
  • Saya menambah algoritma terbaru.
  • Saya memperbaiki algoritma 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 docstring 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 = linux
  • Go version = go 1.21.3

slowy07
slowy07 approved these changes Oct 14, 2023
View reviewed changes
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, thanks for contribution @malvinval !

@slowy07 slowy07 merged commit 4b75a3f into bellshade:main Oct 14, 2023
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@slowy07 slowy07 slowy07 approved these changes

@nuraskiah nuraskiah Awaiting requested review from nuraskiah nuraskiah is a code owner

Assignees
No one assigned
Labels
HACKTOBERFEST-ACCEPTED markdown files sudah dirieview!
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@malvinval @slowy07