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

feat: convert extension html to md #1769

Merged
merged 8 commits into from
Aug 14, 2021
Merged

feat: convert extension html to md #1769

merged 8 commits into from
Aug 14, 2021

Conversation

hochan222
Copy link
Member

#1698 issue 에 대한 후속 조치

@github-actions github-actions bot added the l10n-ko Issues related to Korean content. label Jul 28, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Jul 28, 2021

Preview URLs

Flaws

Note! 3 documents with no flaws that don't need to be listed. 🎉

URL: /ko/docs/Web/JavaScript/Equality_comparisons_and_sameness
Title: 동치 비교 및 동일성
on GitHub
Flaw count: 6

  • macros:
    • /ko/docs/Web/JavaScript/Reference/Global_Objects/Math/atan2 does not exist but fallbacked on /en-US/docs/Web/JavaScript/Reference/Global_Objects/Math/atan2
  • broken_links:
    • Can use the English (en-US) link as a fallback
    • Can use the English (en-US) link as a fallback
    • Can use the English (en-US) link as a fallback
    • Can use the English (en-US) link as a fallback
  • heading_links:
    • h2 heading contains an <a> tag

URL: /ko/docs/Web/JavaScript/A_re-introduction_to_JavaScript
Title: JavaScript 재입문하기 (JS ​튜토리얼)
on GitHub
Flaw count: 18

  • broken_links:
    • Can't resolve /ko/docs/docs/Web/JavaScript/Data_structures#bigint_type
    • Can't resolve /ko/docs/docs/Web/JavaScript/Data_structures#undefined_type
    • Can't resolve /ko/docs/docs/Web/JavaScript/Data_structures#null_type
    • Can't resolve /ko/docs/docs/Web/JavaScript/Data_structures#bigint_type
    • Can't resolve /ko/Core_JavaScript_1.5_Reference/Operators/Arithmetic_Operators
    • and 13 more flaws omitted

URL: /ko/docs/Web/JavaScript/EventLoop
Title: 동시성 모델과 이벤트 루프
on GitHub
Flaw count: 2

  • broken_links:
    • Can use the English (en-US) link as a fallback
    • Can use the English (en-US) link as a fallback

URL: /ko/docs/Web/JavaScript/Data_structures
Title: 자바스크립트의 자료형
on GitHub
Flaw count: 19

  • macros:
    • /ko/docs/Web/JavaScript/Reference/Map does not exist
    • /ko/docs/Web/JavaScript/Reference/Int8Array does not exist
    • /ko/docs/Web/JavaScript/Reference/Uint8Array does not exist
    • /ko/docs/Web/JavaScript/Reference/Uint8ClampedArray does not exist
    • /ko/docs/Web/JavaScript/Reference/Int16Array does not exist
    • and 7 more flaws omitted
  • broken_links:
    • Can use the English (en-US) link as a fallback
    • Can use the English (en-US) link as a fallback
    • Can use the English (en-US) link as a fallback
    • Can use the English (en-US) link as a fallback
    • Can use the English (en-US) link as a fallback
    • and 2 more flaws omitted

External URLs

URL: /ko/docs/Web/JavaScript/Equality_comparisons_and_sameness
Title: 동치 비교 및 동일성
on GitHub


URL: /ko/docs/Web/JavaScript/A_re-introduction_to_JavaScript
Title: JavaScript 재입문하기 (JS ​튜토리얼)
on GitHub


URL: /ko/docs/Web/JavaScript/EventLoop
Title: 동시성 모델과 이벤트 루프
on GitHub


URL: /ko/docs/Web/JavaScript/Enumerability_and_ownership_of_properties
Title: Enumerability and ownership of properties
on GitHub

No new external URLs


URL: /ko/docs/Web/JavaScript/Closures
Title: 클로저
on GitHub


URL: /ko/docs/Web/JavaScript/Data_structures
Title: 자바스크립트의 자료형
on GitHub


URL: /ko/docs/Web/JavaScript/About_JavaScript
Title: JavaScript에 대하여
on GitHub

(this comment was updated 2021-07-28 13:10:15.964950)

@hochan222
Copy link
Member Author

hochan222 commented Jul 28, 2021

@alattalatta 혹시 해당 PR 확인 한번 해주실 수 있나요?


앗 이렇게 하면 안되겠네요.. squash merge를 할 수 없으니까.... 일괄적으로 확장자 변환만 진행해야할까요. 음... 방법을 생각할 시간이 필요할것같습니다.

@alattalatta
Copy link
Member

@hochan222 음, 커밋만 보면 지금 괜찮을 것 같아요. 맨 처음 커밋에서 확장자만 바꾸고, 후속 커밋으로 내용 변경하니까요. git log --follow (파일)로 한번 확인해보세요.

에디터 어떤거 쓰시는지는 모르겠지만 VSCode 쓰시면...
image

Toggle Follow Renames 켜놓으시면 사이드바에서 이름 변경 전 히스토리까지 보일거예요.

@hochan222
Copy link
Member Author

hochan222 commented Jul 28, 2021

@alattalatta 답변해주셔서 감사합니다. (저는 vscode를 사용합니다! 앗.. 이런 경우도 있었네요.. 알려주시지 않았으면 정말 큰일이 일어날뻔 했습니다. 감사합니다 ㅠㅠ)

mdn/content#6936 이슈에서 아래 내용이 언급돼있습니다.
개별적으로 진행한다면, markdown PR에 대해서는 rebase and merge를 해야하는데, 만약 PR 주신 분들중에서 commit을 많이 주신다면 문제가 될 수 있으니 따로 가이드라인을 작성할때 관련해서 언급을 하는게 좋을까요?

These are two commits and we must not squash merge.

@alattalatta
Copy link
Member

alattalatta commented Jul 28, 2021

@hochan222 기본 값이 스쿼시인가요? 저는 파일이 많아져도 커밋을 두세 개 정도만 하니까 별 생각을 안해서... 몰랐어요 😅

그러면 기존 파일의 MD 변환을 포함하는 경우에는 커밋 두 개(확장자 변경/내용 변경) 정도로만 올리시도록 안내하고, 리뷰어께서도 스쿼시 하지 않도록 지침을 세우는게 좋아 보이긴 해요. 까다롭네요... 변환 오류 최대한 수정해서 한 번에 바꿔버리는게 편하긴 하겠어요

@hochan222
Copy link
Member Author

hochan222 commented Jul 28, 2021

@alattalatta 같이 고민해주셔서 감사합니다.

기본 값이 스쿼시인가요?

네 ㅠㅠ

그러면 기존 파일의 MD 변환을 포함하는 경우에는 커밋 두 개(확장자 변경/내용 변경) 정도로만 올리시도록 안내하고, 리뷰어께서도 스쿼시 하지 않도록 지침을 세우는게 좋아 보이긴 해요. 까다롭네요... 변환 오류 최대한 수정해서 한 번에 바꿔버리는게 편하긴 하겠어요

한꺼번에 수정하기 전까지는... 맞아요 ㅠㅠ 만약 PR에서 수정사항이 발생한다면.. ㅠㅠ 우선, 지침부터 만들도록 하겠습니다.

@hochan222 hochan222 requested a review from yechoi42 July 28, 2021 14:03
@hochan222 hochan222 marked this pull request as ready for review July 28, 2021 14:03
@hochan222 hochan222 requested a review from a team as a code owner July 28, 2021 14:03
@igy95
Copy link
Member

igy95 commented Aug 2, 2021

@hochan222 이슈와 pr 모두 확인 마쳤는데요, 제가 병합 진행하면 될까요 호찬님? 👀

@hochan222
Copy link
Member Author

hochan222 commented Aug 2, 2021

@hochan222 이슈와 pr 모두 확인 마쳤는데요, 제가 병합 진행하면 될까요 호찬님? 👀

@GwangYeol-Im 네! 병합해주세요 :)

@igy95 igy95 merged commit d2439a3 into mdn:main Aug 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-ko Issues related to Korean content.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants