-
Notifications
You must be signed in to change notification settings - Fork 92
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
docs: fix typo #523
docs: fix typo #523
Conversation
말씀해주신 것처럼 it이 가르키는 것이 |
혹시 영문도 같이 수정해주실 수 있나요? |
네 영문도 수정했습니다! |
@@ -40,7 +40,7 @@ bookRepository.findPage(pageable) { | |||
## Spring Batch | |||
|
|||
SpringBatch는 JPQL로 쿼리를 할 수 있도록 `JpaPagingItemReader`와 `JpaCursorItemReader`를 제공합니다. | |||
Kotlin JDSL은 DSL로 생성된 JPQL 쿼리가 JpqReader들에서 실행될 수 있도록 `KotlinJdslQueryProvider`을 제공합니다. | |||
Kotlin JDSL은 DSL로 생성된 JPQL 쿼리가 위 ItemReader들에서 실행될 수 있도록 `KotlinJdslQueryProvider`을 제공합니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
제 생각엔 KotlinJdslQueryProvider
을 보단 KotlinJdslQueryProvider
를 표현이 좀 더 자연스러운것 같아요!
추가로 동일 문서상의 Spring Boot AutoConfigure 파트에서 KotlinJdslAutoConfiguration
를 보단 KotlinJdslAutoConfiguration
을 표현이 자연스러운것 같아요~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
을 -> 를
, 를 -> 을
이군요. 말씀해주신 게 더 자연스럽네요. @jbl428 님 혹시 이 부분도 같이 수정해주실 수 있으신가요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
해당 부분도 반영했습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
문서까지 같이 확인해주시고 수정해주셔서 감사합니다!
Modifications
Kotlin JDSL provides KotlinJdslQueryProvider so that a JPQL query created in DSL can be executed on it.
인데 마지막 it이 them 이 되어야 하는건지 확실하지 않네요Commit Convention Rule
commit type
please describe it on the Pull Request