Skip to content

[Android] PR 피드백 수정 #41

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

Jump to bottom
Merged
merged 3 commits into from
Apr 12, 2022
Merged

[Android] PR 피드백 수정 #41

merged 3 commits into from
Apr 12, 2022

Conversation

banjjak2
Copy link
Collaborator

📝 Description

  1. 변수명 카멜 케이스로 변경
  2. History 대신 Histories로 메소드 명 변경
  • 메소드 이름으로 명확하게 어떤 기능을 하는 것인지 알 수 있도록 변경하였음
  1. styles.xml 의 item 한 줄로 변경
  • 일반적인 컨벤션을 따르도록 변경

banjjak2 added 3 commits April 12, 2022 11:22
@banjjak2
refact: 피드백 적용 - 카멜 케이스로 변수명 변경
588eab6
@banjjak2
chore: 피드백 적용 - History 대신 Histories로 메소드 명 변경
fcbff7b 
메소드 이름으로 명확하게 어떤 기능을 하는 것인지 알 수 있도록 변경하였음
@banjjak2
chore: 피드백 적용 - styles.xml 의 item 한 줄로 변경
fa16272 
일반적인 컨벤션을 따르도록 변경
@banjjak2 banjjak2 added the Android 안드로이드 label Apr 12, 2022
@banjjak2 banjjak2 self-assigned this Apr 12, 2022
@banjjak2 banjjak2 merged commit c7119d3 into develop-Android Apr 12, 2022
@banjjak2 banjjak2 deleted the feature/Android/pr branch April 12, 2022 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@banjjak2 banjjak2

Labels

Android 안드로이드

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@banjjak2