-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Conversation
Preview URLs
FlawsNote! 3 documents with no flaws that don't need to be listed. 🎉 URL:
URL:
URL:
URL:
External URLsURL:
URL:
URL:
URL: No new external URLs URL:
URL:
URL:
(this comment was updated 2021-07-28 13:10:15.964950) |
feat: convert html to markdown, Enumerability_and_ownership_of_properties
@alattalatta 혹시 해당 PR 확인 한번 해주실 수 있나요? 앗 이렇게 하면 안되겠네요.. squash merge를 할 수 없으니까.... 일괄적으로 확장자 변환만 진행해야할까요. 음... 방법을 생각할 시간이 필요할것같습니다. |
@hochan222 음, 커밋만 보면 지금 괜찮을 것 같아요. 맨 처음 커밋에서 확장자만 바꾸고, 후속 커밋으로 내용 변경하니까요. 에디터 어떤거 쓰시는지는 모르겠지만 VSCode 쓰시면... Toggle Follow Renames 켜놓으시면 사이드바에서 이름 변경 전 히스토리까지 보일거예요. |
@alattalatta 답변해주셔서 감사합니다. (저는 vscode를 사용합니다! 앗.. 이런 경우도 있었네요.. 알려주시지 않았으면 정말 큰일이 일어날뻔 했습니다. 감사합니다 ㅠㅠ) mdn/content#6936 이슈에서 아래 내용이 언급돼있습니다.
|
@hochan222 기본 값이 스쿼시인가요? 저는 파일이 많아져도 커밋을 두세 개 정도만 하니까 별 생각을 안해서... 몰랐어요 😅 그러면 기존 파일의 MD 변환을 포함하는 경우에는 커밋 두 개(확장자 변경/내용 변경) 정도로만 올리시도록 안내하고, 리뷰어께서도 스쿼시 하지 않도록 지침을 세우는게 좋아 보이긴 해요. 까다롭네요... 변환 오류 최대한 수정해서 한 번에 바꿔버리는게 편하긴 하겠어요 |
@alattalatta 같이 고민해주셔서 감사합니다.
네 ㅠㅠ
한꺼번에 수정하기 전까지는... 맞아요 ㅠㅠ 만약 PR에서 수정사항이 발생한다면.. ㅠㅠ 우선, 지침부터 만들도록 하겠습니다. |
@hochan222 이슈와 pr 모두 확인 마쳤는데요, 제가 병합 진행하면 될까요 호찬님? 👀 |
@GwangYeol-Im 네! 병합해주세요 :) |
#1698 issue 에 대한 후속 조치