search

snu-sf-class / swpp202401

Principles and Practices of Software Development Main Repository
14 stars 4 forks source link

[Project] Graded PR에 대한 질문 #115

Open flylet222 opened 6 months ago

flylet222 commented 6 months ago

Project Documentation에 적혀있는

[4/25 Update] Each student should merge at least one pull request per sprint. ○ [5/2 Update] Only ‘graded’ PRs counts; otherwise, we cannot grade your performance on a sprint and have to give you zero points.

에서의 'graded'가 어떤 의미인지 궁금합니다. 문서에 다른 곳에는 grade라는 표현이 설명되어있지 않아서 정확한 의미를 알고 싶습니다.

감사합니다.

strikef commented 6 months ago

최적화 패스를 직접 구현하는 PR을 말하는 것입니다. 즉, NFC, Update, Revert, Existing pass PR 이 아닌 경우를 이야기합니다. (Reverted 는 포함됩니다!)

mscheong01 commented 6 months ago

안녕하세요, 위 내용과 관련하여 이번 스프린트에 작업한 pass를 다음 스프린트에 이어서 작업하고자 하는데, 해당 PR 을 완전히 작동하지 않는 상태에서 해당 패스를 비활성화시키고 우선 merge 하는 것이 좋을지, 아니면 PR 을 열어두고 다음 스프린트에 마무리하여 merge 하면 될지 질문드립니다.

strikef commented 6 months ago

원하는 만큼 기능이 구현되지는 않았지만 테스트는 통과가 가능한 수준이라면, 그리고 이것 외에는 merge할 PR이 도저히 존재하지 않는다면, 말씀하신 대로 비활성화 상태로 merge하셔도 됩니다.

반드시 unit test는 통과가 가능한 수준이어야 합니다!

mscheong01 commented 6 months ago

@strikef 아직 unit test 를 통과하게 해두지 않았지만, 이전에 작성했던 IR pass 의 성능을 최적화한 다른 PR 이 머지 가능해서 이번 스프린트에는 열어두도록 하겠습니다. 확인해주셔서 감사합니다~