[Release] v2.0.7 릴리즈 #233
[Release] v2.0.7 릴리즈 #233
Conversation
Co-authored-by: 나용준 <[email protected]>
Summary of ChangesHello @yongjun0511, 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! 이 풀 리퀘스트는 v2.0.7 릴리즈를 위한 변경사항을 포함합니다. 주요 변경사항은 파트너십 정보 표시의 정확성을 높이고, 파트너십이 사용자에게 노출되는 기간을 확장하여 최신 파트너십 정보가 더 오랫동안 유지되도록 개선하는 것입니다. Highlights
Using Gemini Code AssistThe 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 by creating a comment using either
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 Limitations & Feedback Gemini Code Assist 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. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. 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. 코드 흐름 따라 릴리즈 준비 새 단장 버그는 안녕 Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
| or (pc is null and pd is null) | ||
| ) | ||
| and p.startDate <= current_date | ||
| and p.startDate >= current_date - 7 |
There was a problem hiding this comment.
JPQL 쿼리 내에 매직 넘버 7이 사용되었습니다. 이 값은 '최근 제휴를 보여주기 위한 기간'이라는 의미를 가지는 것으로 보입니다. 이처럼 의미를 가지는 숫자는 상수로 정의하여 사용하면 코드의 가독성과 유지보수성을 높일 수 있습니다. 예를 들어, PARTNERSHIP_LOOKBACK_DAYS와 같은 이름의 상수로 관리하는 것을 고려해 보세요.
| LEFT JOIN FETCH p.partnershipCollege | ||
| LEFT JOIN FETCH p.partnershipDepartment | ||
| WHERE p.startDate <= CURRENT_DATE and (p.endDate is null or p.endDate >= CURRENT_DATE)""") | ||
| WHERE p.startDate >= current_date - 7 and (p.endDate is null or p.endDate >= CURRENT_DATE)""") |
There was a problem hiding this comment.
이 쿼리에는 두 가지 개선점을 제안합니다.
-
매직 넘버 사용:
PartnershipRepository와 마찬가지로 숫자7이 매직 넘버로 사용되었습니다. 이 값을 상수로 추출하여 관리하면 여러 곳에서 동일한 값을 일관되게 사용하고 변경이 필요할 때 한 번에 수정할 수 있어 유지보수성이 향상됩니다. -
SQL 키워드 대소문자 불일치: 쿼리 내에서
current_date와CURRENT_DATE가 함께 사용되고 있습니다. 기능적으로는 문제가 없지만, 코드의 일관성을 위해 하나의 스타일(예: 소문자)로 통일하는 것이 좋습니다.
WHERE p.startDate >= current_date - 7 and (p.endDate is null or p.endDate >= current_date)")
No description provided.