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

Add GitHub Trending #3

Merged
merged 1 commit into from
Sep 6, 2018
Merged

Add GitHub Trending #3

merged 1 commit into from
Sep 6, 2018

Conversation

cruisediary
Copy link
Contributor

  • Add github swift trending in README.md

@giftbott
Copy link
Owner

giftbott commented Sep 4, 2018

안녕하세요. cruisediary 님
GitHub 카테고리를 별도로 추가 하였으니 해당 위치로 옮겨주시면 바로 반영하겠습니다.
설명은 가능하면 한글로 적어주세요
링크 추가해주셔서 감사합니다. :)

@cruisediary
Copy link
Contributor Author

GitHub 카테고리에는 아래 목록과 같이 Github 관련 사이트로 구성되어 있습니다.
이 곳에는 Github Trending (all language)을 추가하면 좋을 것 같습니다.

다만 레포 설명인 iOS 개발자를 위한 링크 모음 취지에 맞게
Swift Github trending(swift language) 링크를 추가하려다 보니
Basic - swift의 Swift GitHub 아래에 두는게 좋을 것 같다고 판단했습니다. :)

@giftbott
Copy link
Owner

giftbott commented Sep 5, 2018

무슨 말씀인지 이해했습니다.

그런데 GitHub 관련 사이트를 꼭 다른 사이트가 아니고 GitHub 와 관련된 데이터를 다루는 링크 라고 생각해보면 Trending 역시 그 중 하나라고 생각해요.
Basic 에는 말 그대로 기본이 되는 링크를 포함시키려 하는데 Trending은 그 대상은 아니라고 판단했어요.
사실 이런 면에서는 swiftLint 도 Basic 에서 빠져야 하는데 카테고리 구성상 넣다 보니... -_-a;

그리고 여전히 Objective-C 를 쓰시는 분들도 계시고
이 iOSDevLinks 도 Unknown languages 에 들어간다는 점을 고려하면
Swift 필터 여부는 크게 상관없을 것 같아요. Trending 사이트 존재만을 알려주어도 괜찮을 듯 합니다.

물론 이건 제 생각이라 전반적으로 더 나은 구성을 할 수 있다면 변경 가능합니다 :)

@cruisediary
Copy link
Contributor Author

Git Awards도 Swift, Objective-C 필터가 가능한 것을 고려하여
all language의 GitHub Trending을 GitHub 카테고리에 추가하였습니다. :)

@cruisediary cruisediary changed the title Add github swift trending Add GitHub Trending Sep 6, 2018
@giftbott giftbott merged commit d31c529 into giftbott:master Sep 6, 2018
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.

2 participants