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

redundant parameter remove #22

Closed
wants to merge 1 commit into from
Closed

Conversation

celiktemha
Copy link

No description provided.

Copy link

@mobx16 mobx16 left a comment

Choose a reason for hiding this comment

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

func WeAre ( takım dizesi ) { kaldır

@mdegis
Copy link

mdegis commented Feb 18, 2019

backward compatibility acisindan baktigimiz zaman, parametreyi kaldirmak yerine go convention'ina uygun olarak blank identifier( _ ) kullanmak daha iyi olabilir.

@@ -2,7 +2,7 @@ package who

import "fmt"

func WeAre(team string) {
Copy link

Choose a reason for hiding this comment

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

👍

@ilkinulas
Copy link
Contributor

Selam Arkadaslar

Reklami izleyenler hatirlayacaktir. WeAre fonksiyonunu multi-line string parametresi ile cagiriyoruz.

weare

Bu reponun amaci reklamda kullandigimiz go kodununun derlenebildigini ve sorunsuz calistirilabilecegini gostermek.

Peak open source projelerine katkida bulunmak isterseniz https://github.com/peakgames adresine goz atabilirsiniz.

@ilkinulas ilkinulas closed this Feb 21, 2019
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.

5 participants