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

기술블로그 오타 수정 #342

Merged
merged 3 commits into from
Mar 13, 2024
Merged

기술블로그 오타 수정 #342

merged 3 commits into from
Mar 13, 2024

Conversation

LimJiGyu
Copy link
Contributor

@LimJiGyu LimJiGyu commented Mar 4, 2024

아주 사소한 문법 오류를 발견하여 바로 수정합니다.

@LimJiGyu LimJiGyu self-assigned this Mar 4, 2024
@LimJiGyu LimJiGyu requested a review from XiNiHa as a code owner March 4, 2024 02:59
Copy link

vercel bot commented Mar 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
developers ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 13, 2024 10:21am

@XiNiHa
Copy link
Contributor

XiNiHa commented Mar 4, 2024

앗 근데 린트가 깨졌네요 😢 함 봐주시겠어요?

@LimJiGyu
Copy link
Contributor Author

LimJiGyu commented Mar 4, 2024

앗 근데 린트가 깨졌네요 😢 함 봐주시겠어요?
넵넵 👍 근데 각 린트룰에 대해 이야기 해보고싶은 내용들이 있습니다

  1. remark-lint-no-emphasis-as-heading :
    image
    이 부분을 헤더로 바꾸라는 룰인데, 린트 기능으로서 마크다운 글을 prettify 하기보다는 표현하고 싶은 형태를 제한하는 것 같다는 느낌이 드는데 어떻게 보시나요?

  2. remark-lint-maximum-line-length : 각 라인 수를 80자로 제한하는 것인데 생각보다 제한 수가 적어서 하나의 온전한 문장 단위로 끊기지도 않고 불필요하게 줄만 길어지는 느낌이 듭니다. 없애거나 글자 제한 수를 늘리는 것은 어떨까요?

  3. remark-lint-list-item-spacing : 위 사진과 동일한 부분에서 발생한 린트 룰 에러인데, 공백 라인을 추가하고 나면 글 상에서 공백이 커집니다. 즉, 표현된 형태는 고정되고 코드만 정리되는게 아니라 결과 자체가 공백이 커지게 됩니다. 방법이 있는지 모르겠지만 만약 이렇게 될 수 밖에 없는거라면 문단 간 공백이 과하게 길어지는 것 같은데 어떠신가요?

@XiNiHa
Copy link
Contributor

XiNiHa commented Mar 5, 2024

  • remark-lint-no-emphasis-as-heading은 의도에 맞게 동작하지 않는 상황이 너무 많은 것 같아서 빼는 게 좋을 듯 싶네요 😅 감사합니다!
  • remark-lint-maximum-line-length는 사실 에디터에서 파일을 2개 열었을 때에도 문장이 짤리지 않고 보여지도록 하는 걸 의도로 80자로 걸긴 했는데.... 저도 약간 짧다는 것에는 공감하니 한 100자 정도로 조정해보도록 하겠습니다
  • remark-lint-list-item-spacing이 위 상황에서 공백을 늘리는 건 각 리스트 항목(1., 2., 3., ...과 - ... 들)마다 세로 여백이 들어가기 때문이라서, 필수적으로 필요한 상황이 아니면 리스트 사용을 자제하는 식으로 문제를 해결하는 것이 맞아 보입니다...!

@XiNiHa
Copy link
Contributor

XiNiHa commented Mar 5, 2024

main에 위 수정사항 반영되어 리베이스해주시면 될 것 같습니다!

src/content/blog/2024-02/v2-zio.mdx Outdated Show resolved Hide resolved
src/content/blog/2024-02/v2-zio.mdx Outdated Show resolved Hide resolved
src/content/blog/2024-02/v2-zio.mdx Outdated Show resolved Hide resolved
src/content/blog/2024-02/v2-zio.mdx Outdated Show resolved Hide resolved
src/content/blog/2024-02/v2-zio.mdx Show resolved Hide resolved
@XiNiHa XiNiHa merged commit feba8a8 into main Mar 13, 2024
4 checks passed
@XiNiHa XiNiHa deleted the fix/blog-typo branch March 13, 2024 10:40
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.

2 participants