Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chore/fix pr labeler #16

Merged
merged 4 commits into from
Jul 28, 2024
Merged

Chore/fix pr labeler #16

merged 4 commits into from
Jul 28, 2024

Conversation

WooWan
Copy link
Contributor

@WooWan WooWan commented Jul 28, 2024

pr labeler github action 수정 테스트

@WooWan
Copy link
Contributor Author

WooWan commented Jul 28, 2024

pull_request_target으로 설정되어 있어, 베이스 브랜치(main) 으로 설정되어 있어 labeler 액션이 여전히 실패하고 있는 것으로 보이고 있습니다.
pull_request로 설정하여 현재 헤드 브랜치(chore/fix-pr-labeler)에서 테스트하였을 때, action이 통과하는 것을 확인하였습니다.

on:
  - pull_request_target
jobs:
  labeler:
    permissions:
      contents: read
      pull-requests: write
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
        with:
          sparse-checkout: |
            .github
      - uses: actions/labeler@v5
        with:
          repo-token: "${{ secrets.GITHUB_TOKEN }}"
          configuration-path: "./.github/pr-labeler-config.yaml"
          configuration-path: ".github/pr-labeler-config.yaml"

Comment on lines +12 to +15
- uses: actions/checkout@v4
with:
sparse-checkout: |
.github
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checkout 액션을 이용하여 configuration-path에 지정된 파일 경로의 파일을 가져올 수 있도록 추가합니다.
참고: chore/fix-pr-labeler

Copy link
Collaborator

@bepyan bepyan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오타 수정 + 오류 대응 감사합니다 👍🏻👍🏻

@WooWan WooWan merged commit 70ccc25 into main Jul 28, 2024
3 of 5 checks passed
@WooWan WooWan deleted the chore/fix-pr-labeler branch July 28, 2024 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants