Conversation
|
해당 브랜치의 이름에 8이 의미가 있나요!?? |
|
기본 브랜치 설정에서 다른 서브브랜치를 풀리퀘스트를 강제머지를 막을 수 있나요? |
|
룰정하는것 gpt에게 물어봐서 나오는것 확인했습니다. 수정토록하겠습니다. |
|
|
||
| public class Manager : MonoBehaviour | ||
| { | ||
| public void OnStart() => Debug.Log("Start!"); |
There was a problem hiding this comment.
함수는 우리 회사의 코드컨벤션 룰 상 { } 로 가야 할 것 같습니다.
There was a problem hiding this comment.
class naming이 중복될 우려가 있습니다. 다른 이름 부탁드립니다.
|
add single comment와 start a review 이해했습니다. |
tttghost
left a comment
tttghost
left a comment
There was a problem hiding this comment.
코드컨벤션 상 수정이 필요해 보이나 기능상의 문제가 없기때문에 저는 패스하겠습니다.
|
|
||
| public class Manager : MonoBehaviour | ||
| { | ||
| public void OnStart() => Debug.Log("Start!"); |
|
와... 한 사이클 돌렸습니다. |
|
close #8 을 하게되면 pullrequest가 완료 될 시에 자동으로 issue도 close가 되는 것인가요? |
|
이 리뷰를 통해 결정해야 하는 것 Code style, One line code to lambda expression.한 줄 코드는 람다 표현식 쓸지, block body로 쓸지는 정하면 됩니다. Merge actioner회사마다 룰이 다른데 |
2 participants
해당 이슈에 설계한 내용대로 구현 및 반영
리뷰 하면서 물어보고 싶은 거 물어보셔도 됩니다.
close #8