Skip to content

[정규 표현식]대주제 번역 수정 #577

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

adk1245
Copy link
Contributor

@adk1245 adk1245 commented Jun 13, 2020

Pull Request 체크리스트

TODO

  • 번역 규칙을 확인하셨나요?
    • 줄 바꿈과 단락을 '원문과 동일하게' 유지하셨나요?
    • 맞춤법 검사기로 맞춤법을 확인하셨나요?
    • 마크다운 문법에 사용되는 공백(스페이스), 큰따옴표("), 작은따옴표('), 대시(-), 백틱(`) 등의 특수문자는 그대로 두셨나요?
  • 로컬 서버 세팅 후 최종 결과물을 확인해 보셨나요?
  • PR 하나엔 번역문 하나만 넣으셨나요?
  • 의미 있는 커밋 메시지를 작성하셨나요?
    • 예시
      • [프락시] 번역
      • [프락시] 과제 번역
      • [if문과 조건부 연산자 '?'] 리뷰
      • [주석] 2차 리뷰
      • [Date 객체와 날짜] 번역

@javascript-translate-bot javascript-translate-bot requested a review from a team June 13, 2020 11:26
@adk1245 adk1245 changed the title [정규 표현식 수정]대주제 번역 수정 [정규 표현식]대주제 번역 수정 Jun 13, 2020
Copy link
Member

@Violet-Bora-Lee Violet-Bora-Lee left a comment

Choose a reason for hiding this comment

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

조사가 빠져서 문장이 불완전해보이는데요,
혹시나 바라보는 관점이 달라 문장 구성 방법도 다를 수 있으니
이견이 있으시면 코멘트 주세요


정규 표현식은 문자열에서 검색과 바꾸기를 할 수 있는 강력한 방법입니다.
문자열에서 검색과 바꾸기를 할 수 있는 강력한 방법 정규 표현식에 대해 알아봅시다.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
문자열에서 검색과 바꾸기를 할 수 있는 강력한 방법 정규 표현식에 대해 알아봅시다.
문자열에서 검색과 바꾸기를 할 수 있는 강력한 방법인 정규 표현식에 대해 알아봅시다.

@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@adk1245
Copy link
Contributor Author

adk1245 commented Jun 17, 2020

원래 의미는 '강력한 방법, 정규 표현식' 이렇게 강조용으로 쓰려다가 쉼표는 안쓰는 것이 좋은 것 같아서 뺐는데 문장이 조금 불안정 해진거같아요. 그래서 조사 넣어서 수정했습니다 확인부탁드립니다!

@Violet-Bora-Lee
Copy link
Member

LGTM! 👍

커밋 하나로 합쳐주시면 바로 머지 진행하겠습니다

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.

3 participants