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

resolver helper들에서 return-camel-case를 언제나 하던 코드 삭제 #31

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ljb7977
Copy link
Contributor

@ljb7977 ljb7977 commented Jun 23, 2022

https://github.com/green-labs/sinsunhi-backend/pull/1777 을 잇는 PR입니다.
몇번 코드가 정리되고 나서 보니 resolver helper 함수들의 맨 마지막에서 언제나 camelCase 변환을 해주고 있는 상황이었고,
그 바깥의 generate-one 함수에서는 return-camel-case? 옵션을 받아서 그에 맞게 camelCase 변환을 해 주기 때문에 resolver helper 함수들에서는 해당 케이스 변환 코드를 모두 삭제해도 될 것 같습니다.

@ljb7977 ljb7977 requested a review from jungwookim June 23, 2022 07:38
Copy link
Contributor

@jungwookim jungwookim left a comment

Choose a reason for hiding this comment

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

👍

@bakyeono
Copy link
Contributor

음~ 리졸버를 정의할 때 EDN을 이용해서 generate-one 을 할 수도 있지만,
resolver-helper를 사용해서 만드는 것도 가능한 걸로 저는 생각을 하고 있었어요.
후처리를 하는 위치가 어디여야 할까 고민이 됩니다.

@ljb7977
Copy link
Contributor Author

ljb7977 commented Jun 23, 2022

실제로 팜모닝브릿지 코드에서 resolver-helper의 함수들을 바로 불러서 설정하고 있긴 한데요, 겉에 defresolver가 감싸져 있고 원래 케이스 변환의 책임은 defresolver에 있었기 때문에 이 함수들에선 제거하는 것이 좋지 않을까 합니다.
(generate-one 함수가 매크로인 defresolver를 함수화한 것이라서, 책임도 그대로 옮겨온 것으로 알고 있습니다)

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.

3 participants