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

infra/DANG-1267: workflow에서 swagger-ui-updates branch에서의 event는 무시하도록 수정 #900

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

do0ori
Copy link
Collaborator

@do0ori do0ori commented Nov 15, 2024

작업 내용 (Content)

  • PR event와 관련된 workflow 파일들에서 swagger-ui-updates에서의 event는 무시하도록 수정

링크 (Links)

https://www.notion.so/do0ori/workflow-swagger-ui-updates-branch-event-13f8ad358684807aa110c19a60760a98

기타 사항 (Etc)

Merge 전 필요 작업 (Checklist before merge)

  • PR 올리기 전 rebase 동기화를 하셨나요?
  • 마지막 줄에 공백 처리를 하셨나요?
  • 커밋 단위를 의미 단위로 나눴나요?
    • 예시
      • 코드 가독성을 위해 메서드를 추출하라
      • if-else 문을 if 문으로 분리하라
      • 불필요한 메서드를 인라인화하라
  • 커밋 본문을 작성하셨나요?
    • 예시
      • 함수는 한 가지 일을 해야 한다는 원칙에 따라 메서드를 추출합니다.
      • if-else는 컴파일 시 처리가 되어 재컴파일 없이 수정 할 수 없습니다.
        이에 따라 코드가 실행되는 순간에 실행이 결정되는 if 문으로 수정합니다.
  • 리뷰 요청 전 Self-Review로 의문점을 해결 하셨나요?
  • PR 리뷰 가능한 크기를 유지하셨나요?
  • CI 파이프라인이 통과가 되었나요?

Updated GitHub workflows to ignore `swagger-ui-updates` branch, ensuring
unnecessary actions are not triggered on this branch.

- Modified `change_ticket_status_done.yaml`, `link_pr_to_ticket.yaml`,
  and `pr_labeler_validator.yaml` to add `branches-ignore` for
  `swagger-ui-updates`.

This update optimizes workflow execution by preventing redundant
actions on auto-generated branches.
Copy link

vercel bot commented Nov 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
dangdang-walk ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 15, 2024 5:06am

@do0ori do0ori merged commit 4ef6909 into main Nov 15, 2024
4 checks passed
@do0ori do0ori deleted the infra/DANG-1267 branch November 15, 2024 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant