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

린트 추가 #51

Merged
merged 4 commits into from
Jul 21, 2024
Merged

린트 추가 #51

merged 4 commits into from
Jul 21, 2024

Conversation

jwson-automation
Copy link
Owner

설명

린트 추가

변경 사항

  • 린트 파일을 추가했습니다. ( analysis_options.yaml )
  • VScode 린트 파일을 추가했습니다. ( settings.json )
  • VScode settings.json 파일을 삭제했습니다.

체크리스트

< 체크리스트 항목을 확인해 주세요. >

  • 오늘도 행복하게 코딩했는가?
  • release 브랜치를 제대로 최신화 하고 이 브랜치에 merge 했는가?
  • PR 제목은 명확하고 간결한가?
  • PR 에는 하나의 작업에 대한 내용만 포함되었는가?
  • 파일명은 누구나 이해할 수 있게 작성되었는가?
  • camelCase를 사용하였는가? (ThisIsCamelCase, this_is_not_camel_case)
  • 텍스트는 AppStrings.dart 파일에서 가져오고 있는가?
  • 디스코드 채팅방에 완료된 태스크를 알렸는가?

스크린샷

< 여기에 이미지/동영상을 드래그 앤 드랍하면 링크가 생성됩니다. >
sample

참조

관련된 이슈나 PR이 있다면 링크를 추가해주세요.

@jwson-automation jwson-automation marked this pull request as ready for review July 20, 2024 04:56
Copy link
Collaborator

@hongmono hongmono left a comment

Choose a reason for hiding this comment

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

LGTM

@jwson-automation jwson-automation merged commit b956831 into release/2024-summer Jul 21, 2024
1 check passed
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