언론사 추가 및 카테고리 업데이트 관련 요청 #7
Conversation
There was a problem hiding this comment.
Hello @benjioh5, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!
Summary of Changes
Hello! Gemini here, providing a summary of this pull request. This PR, authored by benjioh5, serves as an initial inquiry and proposal to expand the collection of media outlets in the repository. The author proposes adding outlets based on lists from the Korean Press Foundation, Daum News partners, and Media Map. The description also raises important points about the need for category updates and establishing rules for handling __all__ regarding non-major or regional news sources if these additions are made. The current changes in this specific PR are limited to adding links to these proposed reference lists within the README.md file.
Highlights
- Reference Sources Added: Links to the proposed sources for finding new media outlets (Korean Press Foundation, Daum News, Media Map) have been added to the README.
- Discussion on Scope and Impact: The PR description initiates a discussion about the necessary follow-up work, including updating categories and defining rules for the
__all__list, should the proposal to add more outlets be accepted.
Changelog
- README.md
- Added links to three external reference lists for potential media outlets: Korean Press Foundation participating media, Daum News partner channels, and Media Map.
Using Gemini Code Assist
The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.
Invoking Gemini
You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.
| Feature | Command | Description |
|---|---|---|
| Code Review | /gemini review |
Performs a code review for the current pull request in its current state. |
| Pull Request Summary | /gemini summary |
Provides a summary of the current pull request in its current state. |
| Comment | @gemini-code-assist | Responds in comments when explicitly tagged, both in issue comments and review comments. |
| Help | /gemini help |
Displays a list of available commands. |
Customization
To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.
Limitations & Feedback
Gemini Code Assist is currently in preview and may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments to provide feedback.
You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.
New links appear,
Pointing to sources so clear,
More news to find.
Footnotes
-
Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩
There was a problem hiding this comment.
Code Review
안녕하세요. PR을 보내주셔서 감사합니다!
먼저, README.md의 참고자료 섹션에 유용한 링크들을 추가해주신 점 좋습니다. 이 링크들은 향후 언론사 목록 확장에 도움이 될 것입니다.
PR 설명에서 문의주신 내용에 대해 답변드립니다:
- 언론사 추가: 한국언론진흥재단 제휴사, 미디어맵, 다음뉴스 제휴 언론사 채널 등재 언론들을 추가하는 작업은 매우 환영합니다! 프로젝트에 큰 도움이 될 것입니다.
- 카테고리 업데이트 및
__all__규칙: 언론사 추가에 따른 카테고리 업데이트와__all__관련 규칙(예: 메이저/마이너 언론사 구분, 지역 신문 처리 등) 정립에 대한 제안 및 작업도 적극적으로 검토하고 지원하겠습니다. 이 부분에 대한 기여도 매우 중요하다고 생각합니다.
현재 PR의 변경 사항(README.md 업데이트)에 대해서는 가독성 향상을 위한 작은 제안이 하나 있습니다. 아래 리뷰 코멘트를 확인해주시면 감사하겠습니다.
앞으로의 기여를 기대하겠습니다!
Summary of Findings
- README.md 참고자료 링크 형식 일관성: 새로 추가된 "다음뉴스 제휴 언론사채널" 링크 설명에 다른 항목들과의 일관성을 위해 "다음뉴스의"와 같은 출처를 명시하는 접두사를 추가하는 것을 고려해볼 수 있습니다. 이는 목록의 가독성을 높이는 데 도움이 됩니다.
Merge Readiness
이 PR은 README.md에 참고자료 링크를 추가하는 유용한 변경을 포함하고 있습니다. 한 가지 작은 제안 사항(링크 설명의 일관성 확보)이 있으며, 이 부분을 수정해주시면 병합 준비가 완료될 것으로 보입니다. PR 설명에 언급된 향후 언론사 데이터 추가 및 카테고리 정비 작업은 이 프로젝트에 매우 긍정적인 기여가 될 것입니다. 해당 작업들을 별도의 PR로 진행해주시면 감사하겠습니다. 저는 이 PR을 직접 승인할 권한은 없지만, 제안된 수정이 반영된다면 병합에 적합하다고 판단됩니다. 다른 리뷰어의 최종 검토 및 승인이 필요할 수 있습니다.
| [대한민국의 텔레비전 방송사 목록](https://ko.wikipedia.org/wiki/%EB%8C%80%ED%95%9C%EB%AF%BC%EA%B5%AD%EC%9D%98_%ED%85%94%EB%A0%88%EB%B9%84%EC%A0%84_%EB%B0%A9%EC%86%A1%EC%82%AC_%EB%AA%A9%EB%A1%9D) | ||
| - 미디어오늘의 [언론링크](http://www.mediatoday.co.kr/com/partners.html) | ||
| - 한국언론진흥재단의 [참여언론사](https://www.kpf.or.kr/front/intropage/intropageShow.do?page_id=08cccf3f3cf549d29f97c04304cab50c) | ||
| - [다음뉴스 제휴 언론사채널](https://news.daum.net/cplist) |
There was a problem hiding this comment.
참고자료 목록의 일관성을 위해 이 항목에도 출처를 명시하는 접두사를 추가하는 것을 고려해 보시겠어요? 예를 들어, 바로 위 항목들은 "미디어오늘의"나 "한국언론진흥재단의"와 같이 출처를 밝히고 있습니다.
이렇게 수정하면 목록 전체의 통일성을 높이고 가독성을 향상시키는 데 도움이 될 것 같습니다.
| - [다음뉴스 제휴 언론사채널](https://news.daum.net/cplist) | |
| - 다음뉴스의 [다음뉴스 제휴 언론사채널](https://news.daum.net/cplist) |
|
말씀하신 방향대로 작업을 해주시겠어요? 고맙습니다 👍🏼 |
안녕하세요.
언론사 RSS를 모아놓은 레포지토리가 없나 하고 검색하다가 발견하여 이렇게 PR 날립니다.
만약 괜찮으시다면, 한국언론진흥재단 제휴사, 미디어맵, 다음뉴스 제휴 언론사채널 등재 언론들을 추가하고 싶은데 괜찮은지 이렇게 먼저 문의드립 니다. (해당 링크 Readme.md에 넣어놨습니다.)
다만, 이럴 경우 카테고리 업데이트가 좀 필요할 듯 하고,
__all__관련해서도 규칙 (e.g. 메이저 언론사가 아닌 언론사들에 대한 처리, 지역신문 등등) 이 필요할 듯 한데 이 부분 제 쪽에서 좀 작업해서 올려도 될까요?