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

[Suggestion]: Is textlint deprecated? #971

Closed
lumirlumir opened this issue Jun 29, 2024 · 7 comments · Fixed by #1048
Closed

[Suggestion]: Is textlint deprecated? #971

lumirlumir opened this issue Jun 29, 2024 · 7 comments · Fixed by #1048
Assignees
Labels
suggestion 새로운 기능 제안에 대한 논의

Comments

@lumirlumir
Copy link
Collaborator

lumirlumir commented Jun 29, 2024

Summary

안녕하세요😊

레포지토리 상의 textlint의 사용이 deprecated 되었나 여쭤보고자 합니다.
(#926 이슈와 유사한 내용입니다.)

Page

No response

Details

textlint를 통한 일관된 용어 사용(Wiki)을 통해 문서 개선을 시도하였지만, 해당 lint의 사용이 레포지토리 상에서 deprecated 되었는지 $ yarn test:textlint 명령어가 사용 불가함을 확인하였습니다. (관련 README도 확인하였습니다.)

판단 근거
  1. package.json 상에 textlint와 관련된 dependencies가 없고, 이에 따라 scripts 상에서도 test:textlint 명령어가 존재하지 않음을 확인하였습니다.
  2. .circleci/config.yml 상에 test:textlint 명령어가 존재하지만, 1번의 이유로 인하여 실행이 불가할 것으로 판단됩니다.
  3. textlint관련 마지막 커밋이 5년 전 입니다.

React 공식 문서를 읽다 보면 Wiki 상에 Translate Glossary로 정리되어있지만, 번역 간 반영이 안되어 있는 경우가 종종 보입니다. 영어를 한글로 번역하다보면, 같은 문맥상의 같은 단어이지만 다르게 해석되는 경우가 종종 발생하는 것 같습니다.

일관된 용어 사용이 문서의 가독성과 전달력을 더욱 높일 수 있을 것 같아 textlint의 사용을 적극 건의하고자 하는데, 혹여 textlint를 재사용할(만약 deprecated 되었다면) 예정이 있으신지 궁금합니다. 만약, 재사용 의사가 있으시다면 package.json, textlint/rules, textlint/tests 등을 수정하여 PR을 하고자 합니다.

추가로, #926 에서 관련 내용을 eslint와 github actions를 통해 자동화 할 때, textlint와 관련된 부분도 추가되었으면 하는 바람입니다.

@lumirlumir lumirlumir changed the title [Suggestion]: Is textlint deprecated? [Suggestion]: Is textlint deprecated? Jun 29, 2024
@hg-pyun
Copy link
Member

hg-pyun commented Jun 30, 2024

#926 이 같은 이슈입니다. 이게 장기간 관리가 안되고있어서.. 한번 업데이트가 필요한 시점이기도합니다. 여유가 되신다면, 작업을 해주시면 매우 좋을 것 같습니다 👍

@lumirlumir
Copy link
Collaborator Author

lumirlumir commented Jun 30, 2024

@hg-pyun

넵! 여유되는대로 작업해서 PR 올리도록 하겠습니다👍👍

아, 그리고 현재 궁금한 점이 몇개 있는데,

  1. 현재 .circleci를 이용하시나요? 사용하지 않으신다면 삭제 후 github actions로 통합하고자 합니다. (개인적인 생각으로는, 한국 및 일본 공식문서에만 존재하는 부분이라, 사용하지 않는다면 삭제하는편이 유지보수에 있어 좀 더 깔끔할 것 같습니다.)
  2. textlint의 README 문서 간 아래와 같은 내용이 있는데, 해당 규칙을 따라야 할까요? 번역 및 오타 수정 등의 PR을 주시는 분들이 모두 지켜야 하는 규칙을 작성하는 것이다보니, 'git pre-commit hook에서만 textlint를 실행'할 뿐만 아니라, PR간 '전체 마크다운 파일을 대상'으로 Checks를 진행하는 편이 좋을 것 같다는 생각입니다.

모든 글이 번역된 상태가 아니며 번역이 완료되어도 새로운 글은 계속해서 번역이 되어야 하기 때문에 git pre-commit hook에서만 textlint를 실행하며 전체 마크다운 파일을 대상으로 CI에서 실행할 계획은 없습니다. 규칙의 구현에 대한 테스트는 CI에서 실행됩니다.

@hg-pyun
Copy link
Member

hg-pyun commented Jul 1, 2024

현재 .circleci를 이용하시나요? 사용하지 않으신다면 삭제 후 github actions로 통합하고자 합니다. (개인적인 생각으로는, 한국 및 일본 공식문서에만 존재하는 부분이라, 사용하지 않는다면 삭제하는편이 유지보수에 있어 좀 더 깔끔할 것 같습니다.)

그녀석도 레거시의 잔재라 지워도 무방할 것 같습니다.

textlint의 README 문서 간 아래와 같은 내용이 있는데, 해당 규칙을 따라야 할까요? 번역 및 오타 수정 등의 PR을 주시는 분들이 모두 지켜야 하는 규칙을 작성하는 것이다보니, 'git pre-commit hook에서만 textlint를 실행'할 뿐만 아니라, PR간 '전체 마크다운 파일을 대상'으로 Checks를 진행하는 편이 좋을 것 같다는 생각입니다.

#963 같이 저희가 번역한 것이 아닌 sync로 올라오는 PR들도 있어서 약간 애매하긴한데.. 걔는 force merge로 어떻게 해결할 수 있을 것 같네요. CI에서 돌리되, 변경된 파일만 체크하는 방식으로 구현하면 조금 더 비용을 아낄 수 있을 것 같습니다.

@lumirlumir
Copy link
Collaborator Author

네 그럼 2번 같은 경우는 가능한 README 맞춰서 진행하도록 하겠습니다. 👍

@lumirlumir
Copy link
Collaborator Author

lumirlumir commented Jul 5, 2024

@hg-pyun 안녕하세요😊 textlint 개발이 거의 마무리 단계에 접어들고 있습니다.

개발 중 Translate Glossary의 일부를 수정 및 보완하였는데, 해당하는 코드를 통해 Translate Glossary 문서 생성을 자동화 하고자 합니다.
이를 통해, 코드를 수정하면 그에 맞춰 수기로 Translate Glossary 문서를 직접 업데이트 해야 하는 비효율성을 없앨 수 있습니다.

하지만, 제가 ko.react.dev에 대한 권한이 없어, gitbook 혹은 github wiki를 통한 문서 생성 자동화를 할 수 없는 상태입니다.🥲

그래서 혹시 ko.react.dev에 대한 권한을 받을 수 있을지 여쭙니다.

@hg-pyun
Copy link
Member

hg-pyun commented Jul 7, 2024

@lumirlumir 권한은 제가 드릴 수 있는게 아니긴 합니다 흑흑.. 개인적으로는 유저에 귀속되지 않도록 Wiki가 아닌 README.md처럼 코드 레벨로 관리하면 어떨까요? 버전 히스토리도 남고, 누구나 기여할 수 있는 형태가 될 것 같아서 이 편이 훨신 더 접근성이 좋을 것 같습니다.

예시) https://github.com/reactjs/ko.react.dev/blob/main/UNIVERSAL-STYLE-GUIDE.md

@lumirlumir
Copy link
Collaborator Author

lumirlumir commented Jul 7, 2024

@hg-pyun 좋은 답변 감사드립니다😊

혹시 권한 획득이 불가할 경우, 위와 같이 진행하는게 좋을 것 같아서, 언급해주신 것과 같은 방식의 대안을 고려해두고 있었습니다.

우선, textlint PR 이후에, wiki 자동 관리 PR도 올리도록 하겠습니다. 그리고 앞선 2개가 모두 완료되면, CI 환경에서의 자동화 PR까지 진행하도록 하겠습니다.👍

@hg-pyun hg-pyun added the suggestion 새로운 기능 제안에 대한 논의 label Jul 8, 2024
hg-pyun added a commit that referenced this issue Jul 13, 2024
* chore: textlint basic environment settings

textlint 관련 기본 환경 설정 진행.

textlintrc.js -> textlintrc: .eslintrc 및 .prettierrc와의 통일성을 위해, .js 확장자를 사용하지 않는 json 형식의 파일로 변경.
package.json에 textlint 및 textlint-filter-rule-comments 추가.
yarn.lock 파일은 package.json 파일에 따라 자동으로 변경됨.

* feat: basic implementation of translateGlossary.js

translateGlossary.js 기본 틀/구조 구현. (아직 디테일한 버그들은 잡지 못한 상태.)
이외, legacy 파일들 삭제. (legacy 파일에서 필요한 부분은 따로 정리하여 보관 중.)

* chore: textlint testing environment settings

textlint testing 관련 환경 설정 진행.

package에 mocha 및 textlint-tester 추가.

* test: new tests about strip.js

./textlint/utils/strip.js에 포함된 모든 함수에 대한 테스트 작성.

* test: new tests about is.js

./textlint/utils/is.js에 포함된 모든 함수에 대한 테스트 작성.

* test: new tests about errMsg.js

./textlint/utils/errMsg.js에 포함된 모든 함수에 대한 테스트 작성.

* feat: new rule translateGlossary.js

translateGlossary 규칙 구현 완료.

* fix: incorrect regex in translateGlossary.js

잘못된 정규표현식 표현 수정.

* test: new tests about translateGlossary.js

./textlint/rules/translateGlossary.js에 대한 테스트 작성.

* rename: textlint/data/tests -> textlint/data/utils

디렉토리 이름 변경.

* fix: correct path handling for textlint/tests/utils

모듈 불러오기 경로가 잘못되어, 수정 진행.

* refactor: change the structure of textlint/data/rules/translateGlossary.js

확장성을 위해 데이터 구조 변경.

* refactor: change the structure of textlint/data/rules/translateGlossary.js

확장성을 위해 데이터 구조 변경.

* feat: add genTranslateGlossaryDocs.js

translate-glossary.md 문서를 자동 생성해주는 generator 도입.
생성된 문서는 repository에 올라갈 필요가 없으므로, .gitignore에 해당 부분 추가.
textlint/README.md는 우선 삭제. 추후에 wiki 디렉토리에 추가 예정.

* rename: textlint/README.md -> wiki/textlint/what-is-textlint.md

파일 이름 및 디렉토리 변경 진행.

* feat: new scripts for textlint

textlint와 관련된 test, docs, lint scripts 추가.

---------

Co-authored-by: Haegul Pyun <phg2491@gmail.com>
lumirlumir added a commit to lumirlumir/fork-ko.react.dev that referenced this issue Jul 23, 2024
textlint translateGlossary 용어 추가.
hg-pyun pushed a commit that referenced this issue Jul 25, 2024
textlint translateGlossary 용어 추가.
lumirlumir added a commit that referenced this issue Sep 1, 2024
docs(wiki): manage wiki at the code level
lumirlumir added a commit that referenced this issue Sep 22, 2024
…nt-staged` (#926 #971) (#1048)

* ci: lint automation using github workflow textlint_lint.yml and lint-staged

* ci: update pull_request event to textlint_lint.yml

* fix(convention): sync convention with other workflows
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
suggestion 새로운 기능 제안에 대한 논의
Projects
None yet
2 participants