Skip to content

강남구에서 일 잘하는 5가지 규칙

김범우 edited this page Sep 26, 2023 · 1 revision
❗ **한 달 동안 레이팀 최우선 순위는 팀 프로젝트 완성이다**
  1. 모든 Slack 메세지는 투명하게 공개한다

    • 업무 상 일은 DM으로 하지 않는다
    • Slack의 알람은 항상 ON 해 놓는다
      • 코어 타임에는 바로 답을 하되 코어 타임이 아닐 경우 지금 당장 답변이 힘들다면 👀 (+ 30분 이내) 눈 이모지를 남긴다
      • 댓글에 리마인더를 걸어둔다
  2. 리뷰에 질문(단순 궁금증)하지 말고 질문은 slack을 통해 한다

    • e.g) @DynamicUpdate 왜 다셨어요? (단순 궁금증) → 선 공부 후 슬랙

    • e.g) 더 나은 구조나 방법 제안 → 리뷰

      → 의견 차이는 Slack에 투명하게 공유한다

  3. 팀 별로 스크럼 마스터를 선출하고, 팀 진행 사항에 대해서는 매일 데일리 스크럼때 공유한다

  4. **2인 이하 30분 이상 고민하지 않는다 → 인원 제한 없음. 그냥 30분이 MAX**

  5. ✨말은 문자 그대로 해석한다 ✨

Github Issue & PR Convention

Issue

💡 **제목** 간단하게 기능만 ex) 로그인 기능

Body

  1. 유저 스토리(백로그)
  2. 세부 사항 구현

Label 달기 FEAT, FIX

Pull Request

💡 📌 기능 설명

📢 PR 포인트

  • main

    • 도메인(엔티티) 클래스, 인터페이스, enum 클래스 중괄호 시작시 공백라인, 필드 사이 공백라인, 마지막 공백라인, 파일 마지막 공백라인
    class MyClass {
    	// 클래스 첫 줄 공백라인
      @Id
      @GeneratedValue
    	private int id;
    
    	public int getId() {
    		return this.id;
    	}
    	// 클래스 마지막 줄 공백라인
    }
    // 파일 마지막 줄 공백라인
  • test

    • 메소드명: successLogin / failLoginWithBlankId
    • @DisplayName(”성공적으로 유저가 생성된다”), @DisplayName(”빈문자열을 넣어 로그인이 실패한다”)
    • BDD 사용
    @Test
    @DisplayName()
    class void test() {
    	// given
    	User user = new User();
    
    	// when
    	User persistUser = userRepository.save(user);
    
    	// then
    	assertEqual(persistUser, user);
    }
    • test 메소드 당 assert 단 하나
  • 아키텍처

    • controller ↔ service : 도메인
  • java convention (camelCase) 따라주세요

  • 변수명에 자료형 붙이지 않기 (userList → users, orderMap → orders 등)

  • Dto 같은 어미 붙이지 않기