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

Feature/#085 auto_merge_approved_pr.yml 오류 수정 #92

Merged
merged 3 commits into from
Nov 11, 2024

Conversation

minjungw00
Copy link
Collaborator

@minjungw00 minjungw00 commented Nov 11, 2024

📝 변경 사항

🔍 변경 사항 설명

Run actions/github-script@v6
ReferenceError: steps is not defined
    at eval (eval at callAsyncFunction (/home/runner/work/_actions/actions/github-script/v6/dist/index.js:15143:16), <anonymous>:3:23)
    at callAsyncFunction (/home/runner/work/_actions/actions/github-script/v6/dist/index.js:15144:12)
    at main (/home/runner/work/_actions/actions/github-script/v6/dist/index.js:15236:26)
    at /home/runner/work/_actions/actions/github-script/v6/dist/index.js:15217:1
    at /home/runner/work/_actions/actions/github-script/v6/dist/index.js:15268:3
    at Object.<anonymous> (/home/runner/work/_actions/actions/github-script/v6/dist/index.js:15271:12)
    at Module._compile (node:internal/modules/cjs/loader:1358:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1416:10)
    at Module.load (node:internal/modules/cjs/loader:1208:32)
    at Module._load (node:internal/modules/cjs/loader:1024:12)
Error: Unhandled error: ReferenceError: steps is not defined

1차 수정

  • returncore.setOutput으로 통일
  • 스크립트에 변수가 제대로 들어갈 수 있도록 수정 ${{ steps.pr.outputs.result }}

2차 수정

  • actions/github-script에서 result가 이미 예약된 변수명임(스크립트의 return값)
  • 따라서 core.setOutput로 작성했던 스크립트를 다시 return으로 수정함
  • Get Pull Request와 Check Approvals로 분리된 step을 하나로 통합함

🙏 질문 사항

  • 리뷰어에게 부탁하고싶은 체크리스트를 추가합니다.

📷 스크린샷 (선택)

  • UI 변경이 있는 경우 스크린샷이나 GIF를 첨부합니다.

✅ 작성자 체크리스트

  • Self-review: 코드가 스스로 검토됨
  • Unit tests 추가 또는 수정
  • 로컬에서 모든 기능이 정상 작동함
  • 린터 및 포맷터로 코드 정리됨
  • 의존성 업데이트 확인
  • 문서 업데이트 또는 주석 추가 (필요 시)

@minjungw00 minjungw00 self-assigned this Nov 11, 2024
@minjungw00 minjungw00 added Bug 기능 또는 UI에서 발생한 오류나 예상치 못한 동작을 해결 Build 개발 환경 세팅(eslint, 컨벤션 등) 또는 패키지 매니저 수정 labels Nov 11, 2024
Copy link
Member

@pipisebastian pipisebastian left a comment

Choose a reason for hiding this comment

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

환경설정 너무 조씁니다~~!👍 고생하셨습니다!!

Copy link
Collaborator

@Ludovico7 Ludovico7 left a comment

Choose a reason for hiding this comment

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

확인했습니다! 고생하셨습니다!

Copy link
Collaborator

@hyonun321 hyonun321 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 !!

@hyonun321 hyonun321 merged commit 9b4b32e into dev Nov 11, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 기능 또는 UI에서 발생한 오류나 예상치 못한 동작을 해결 Build 개발 환경 세팅(eslint, 컨벤션 등) 또는 패키지 매니저 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants