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

Refactor : 테스트 제거 [임시] #29

Merged
merged 2 commits into from
Nov 11, 2023

Conversation

jobkaeHenry
Copy link
Collaborator

No description provided.

@jobkaeHenry jobkaeHenry merged commit c1c622a into main Nov 11, 2023
1 of 2 checks passed
@jobkaeHenry jobkaeHenry deleted the 글작성-페이지-고도화 branch November 11, 2023 21:47
@@ -42,7 +41,6 @@ export default function RootLayout({
overflow: "auto",
}}
>
<MSWInit></MSWInit>
{children}
</Box>
<NavigationBar />

Choose a reason for hiding this comment

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

위 코드 패치를 간략히 코드 리뷰해 드리겠습니다. 버그 위험 및 개선 제안이 있다면 환영합니다:

  1. import MSWInit from "@/components/mock/MSWInit";<MSWInit></MSWInit>을 제거한 후에도 결과에 영향을 미치지 않는다면, 이 코드는 삭제할 수 있습니다.
  2. Pretendard 폰트의 임포트 부분에서 ~/assets/font/Pretendard 경로를 확인하여 절대 경로 또는 상대 경로로 변경하는 것이 좋습니다.
  3. import { Box, GlobalStyles } from "@mui/material"; 문장 뒤에 개행이 필요합니다.

개선 제안을 참고하시기 바랍니다.

@@ -29,6 +29,7 @@
// quoteCount: 0,
// likedByMe: false,
// followedByMe: false,

// };

// const meta = {

Choose a reason for hiding this comment

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

주어진 코드 패치에 대해 간단한 코드 리뷰를 해 드리겠습니다. 버그 위험과 개선 제안을 환영합니다.

이 코드 패치에서는 등장하지 않으나 특정 질문에 대한 답변으로서, 주석에서 meta 객체가 초기화되지 않았습니다.

개선 제안:
const meta = {} 형태로 meta 객체를 초기화하는 것이 좋습니다. 이렇게 하면 나중에 해당 객체 속성을 추가해야 할 경우 초기 값이 설정되어 있을 겁니다.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

[Unit test] 유닛테스트를 통과하지 못했습니다.

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.

1 participant