[Fix] delete merged branch github action

[hanseulhee]

안녕하세요! 코드잇 스프린트 1기를 수강 중인 한슬희라고 합니다.

문제

위클리 미션을 진행하면서 기존에 있던 delete-merged-branch-config.yml이 작동하지 않았습니다.

해결

먼저 .github/delete-merged-branch-config.yml이 었던 구조를 .github/workflows/delete-merged-branch-config.yml로 변경하였습니다.

SvanBoxel/delete-merged-branch에서 secret GITHUB_TOKEN을 확인하라 되어있었으나 기존 파일에서는 되어있지 않았습니다.

이에 해당 사항을 수정하였습니다.

jobs:
  delete-branch:
    runs-on: ubuntu-latest
    steps:
      - name: delete branch
        uses: SvanBoxel/delete-merged-branch@main
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

사용 권한을 위해 아래와 같이 permissions을 작성하신 것 같은데 GITHUB_TOKEN의 권한 문제 때문에 동작이 안되는 에러가 있었습니다.

permissions:
  pull-requests: write

Action failed with error: Resource not accessible by integration

관련 issue에서 permission은 필요하지 않다고도 나와 있어 이를 삭제하였습니다.

궁금한 점

레포에서 Settings > General 에 아래와 같이 머지 후 브랜치를 자동으로 삭제해주는 기능을 깃헙에서 제공하는데 따로 github actions을 사용하신 이유가 있으신지 궁금합니다!

기타

머지 후 브랜치를 직접 삭제해왔었는데 이번 기회를 통해 많이 배울 수 있었습니다 🙂

혹시나 제가 잘못 알고 있는 부분이 있다면 말씀 부탁드립니다 🔥🫡

감사합니다!

[withyj-codeit]

• fork한 내용을 머지하는 방식이 아닌, 파생된 브랜치에 PR로 머지할 때(part1 <- part-week1) Github 설정만으로 삭제되지 않는 경우가 있어(일시적인 오류였는지, branch protection룰 하고 충돌해서인지 원인이 무엇인지 확인하진 못했어요) 해당 설정을 추가했었는데, Github 설정만으로 삭제되면 해당 설정은 필요 없어요.
• 현재 fork한 레포에서 작업한 브랜치를 pr머지하는 방식이라 작업중인 브랜치가 남을 일이 없어 더욱 해당 파일 설정은 필요하지 않을 것 같아요.
• 따라서 해당 파일은 삭제하는 방향으로 수정할게요.

제보해주셔서 감사해요 :)

[hanseulhee]

@withyj-codeit

답변 감사합니다!
말씀해주신 내용을 따로 적용해봤는데 Settings > General 의 머지 후 브랜치를 자동으로 삭제해주는 기능을 codeit-bootcamp-frontend/1-Weekly-Mission에서 사용하더라도 fork한 레포에서 만든 브랜치는 머지가 된 후에도 삭제되지 않는 것을 볼 수 있었습니다.

fork를 했더라면 fork한 레포의 Settings에서 머지 후 브랜치를 자동으로 삭제해주는 기능을 사용해야 해당 기능이 가능하였습니다.

추가적으로 로컬에서 만든 브랜치도 branch protection rule이 적용되어 있다면 해당 기능을 설정했어도 삭제가 자동으로 되지 않습니다.

따라서 github actions를 사용하는 게 맞을 것 같습니다. 🙂

[withyj-codeit]

확인 감사해요~
해당 내용 반영해볼게요