Skip to content

Conversation

@leebs0521
Copy link
Collaborator

resolved :

📌 과제 설명

  • QueryDsl 의존성 추가
  • location 도메인 dsl 적용

✅ PR 포인트 & 궁금한 점

  • LocationRepository 구현체 이름 ~impl 괜찮을지 의견 부탁드립니다.

@leebs0521 leebs0521 self-assigned this Nov 22, 2024
@leebs0521 leebs0521 linked an issue Nov 22, 2024 that may be closed by this pull request
2 tasks
Copy link
Collaborator

@m-a-king m-a-king left a comment

Choose a reason for hiding this comment

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

예시 올려주셔서 감사합니다!


@RequiredArgsConstructor
@Repository
public class LocationRepositoryImpl implements LocationRepository {
Copy link
Collaborator

Choose a reason for hiding this comment

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

저는 좋은 것 같아요!

@sonarqubecloud
Copy link

Copy link
Collaborator

@ayoung-dev ayoung-dev left a comment

Choose a reason for hiding this comment

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

Impl 좋은 거 같습니다!

@leebs0521 leebs0521 merged commit 125efc7 into main Nov 23, 2024
2 checks passed
@leebs0521 leebs0521 deleted the feature/27-add-querydsl branch November 23, 2024 04:21
m-a-king pushed a commit that referenced this pull request Nov 24, 2024
* build(query-dsl): QueryDsl 의존성 추가

* chore: QueryDsl Q클래스 .gitignore 추가

* feat: QueryDsl Config 추가

* feat: location 도메인 queryDsl 적용

* chore: 불필요한 import 제거

* chore: 개행 추가
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] QueryDsl 의존성 추가

5 participants