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.

Sign up for GitHub

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

리뷰 test #113

Closed
wants to merge 1 commit into from
Closed

리뷰 test #113

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions fe/src/app/components/common/Button.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@

import React, { useState } from "react";

// 테스트

export interface ButtonProps {
label: string;
onClick: () => void;
Copy link

Choose a reason for hiding this comment

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

코드를 살펴보았습니다. 여기 몇 가지 코멘트를 드리겠습니다.

  1. 주석 사용: 추가한 // 테스트 주석은 의미가 불분명합니다. 주석은 코드의 의도를 명확히 하는 데 도움이 되어야 하므로, 왜 테스트 주석이 필요한지 구체적으로 설명하는 것이 좋습니다. 예를 들어, // 버튼 테스트를 위해 추가된 주석과 같이 작성하면 좋습니다.

  2. 타입 정의: ButtonProps 인터페이스는 잘 정의되어 있습니다. 하지만 사용되는 컴포넌트가 실제로 있는 경우 이를 추가적으로 보여주면 코드가 더욱 완전해질 수 있습니다. 예를 들어, Button 컴포넌트가 어떻게 ButtonProps를 사용하는지 보여주면 좋습니다.

  3. 주석 위치: 코드의 상단에 주석을 추가하는 것은 유용할 수 있지만, 더 많은 정보가 필요한 경우 주석이 제대로 설명되는 것이 중요합니다. 필요한 부분에 대해 더 구체적으로 설명해 주세요.

추가 코드나 특정 기능에 대한 설명이 있다면 더 나은 피드백을 제공할 수 있을 것입니다. 필요하시면 계속 말씀해 주세요!

Expand Down
Loading