snu-sf-class / swpp202401

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

Policy regarding the project codebase updates #36

Open strikef opened 7 months ago

strikef commented 7 months ago

컴파일러 업데이트를 팀 리포지토리에 적용할 때도 PR을 통해 머지하셔야 합니다. 컴파일러 업데이트를 적용하기 위한 PR은 스프린트 별 PR 수 제한에 포함되지 않고, 리뷰가 필요 없으며, 성적 산정에 사용되지 않습니다. 하지만 CI는 통과하는 상태로 머지하셔야 합니다. 업데이트하실 때는 1) 하나 이상의 업데이트 커밋을 별도 브랜치로 cherry-pick한 이후, 2) 해당 브랜치를 PR로 올리시면 됩니다. 한 번의 Update PR에 2개 이상의 update commit이 포함되어도 괜찮습니다. 만약 추가한 패스 등으로 인해 CMakeList에 사소한 merge conflict가 발생하는 경우는 이를 PR description에 명시하고 conflict를 해소하는 커밋을 추가하셔도 됩니다. C++ 코드는 별도의 공지가 없는 한 수정하면 안 됩니다! 해당 PR은 제목이 [Sprint N, Update] 로 시작하도록 해주시기 바랍니다.

You should create a separate PR to merge the update into your team repository. This PR is exempt from any PR count restriction, does not require review, and will not be used for grading. But it should still pass the CI. When applying the update, you should 1) cherry-pick one or more update commits from the template repo onto a separate branch, and then 2) create a PR from that branch. Each Update PR may contain more than one update commits. In case the update introduces trivial merge conflicts in CMakeList (due to added passes, or etc.), you may add more commits to resolve them. But please describe such changes in PR description. Editing C++ code is not allowed unless notified otherwise! Also, please add [Sprint N, Update] at the beginning of the title.


예외적으로, Sprint 0 동안 업데이트된 항목에 관해서는 PR 없이 머지하셔도 괜찮습니다. During Sprint 0, you may merge the updates to your team repository without creating PR despite the above policy,