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

docs: fix typo #523

Merged
merged 1 commit into from
Nov 13, 2023
Merged

docs: fix typo #523

merged 1 commit into from
Nov 13, 2023

Conversation

jbl428
Copy link
Contributor

@jbl428 jbl428 commented Nov 13, 2023

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 Description
feat New Feature
fix Fix bug
docs Documentation only changed
ci Change CI configuration
refactor Not a bug fix or add feature, just refactoring code
test Add Test case or fix wrong test case
style Only change the code style(ex. white-space, formatting)
chore It refers to minor tasks such as library version upgrade, typo correction, etc.
  • If you want to add some more commit type please describe it on the Pull Request

@shouwn
Copy link
Member

shouwn commented Nov 13, 2023

영어 원문은 Kotlin JDSL provides KotlinJdslQueryProvider so that a JPQL query created in DSL can be executed on it. 인데 마지막 it이 them 이 되어야 하는건지 확실하지 않네요.

말씀해주신 것처럼 it이 가르키는 것이 JpaPagingItemReaderJpaCursorItemReader 이기 때문에 them이 더 맞겠네요. ❤️

@shouwn
Copy link
Member

shouwn commented Nov 13, 2023

혹시 영문도 같이 수정해주실 수 있나요?

@jbl428
Copy link
Contributor Author

jbl428 commented Nov 13, 2023

네 영문도 수정했습니다!

@@ -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`을 제공합니다.
Copy link
Contributor

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을 표현이 자연스러운것 같아요~

Copy link
Member

Choose a reason for hiding this comment

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

을 -> 를, 를 -> 을이군요. 말씀해주신 게 더 자연스럽네요. @jbl428 님 혹시 이 부분도 같이 수정해주실 수 있으신가요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

해당 부분도 반영했습니다!

Copy link
Member

@shouwn shouwn left a comment

Choose a reason for hiding this comment

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

문서까지 같이 확인해주시고 수정해주셔서 감사합니다!

@shouwn shouwn merged commit d9f659b into line:develop Nov 13, 2023
3 checks passed
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