Skip to content

Add 'paths to ignore' to auto-build GitHub workflows - #810

Merged
seokho-son merged 1 commit into
cloud-barista:mainfrom
jangh-lee:main
Sep 28, 2021
Merged

seokho-son merged 1 commit into
cloud-barista:mainfrom
jangh-lee:main

Conversation

@jangh-lee

Copy link
Copy Markdown
Contributor

#797 이슈입니다.

docs/ 디렉토리 아래 파일들을 업데이트시 별도의 빌드테스트 X, 컨테이너 이미지 업데이트 불필요하여
paths-ignore에 docs 디렉토리의 모든 하위폴더에 대해 무시하도록 경로 설정하였습니다.

이렇게 경로 설정된 파일은 2개입니다.
build-amd64-container-image.yaml
publish-multi-arch-container-images.yaml

@seokho-son

Copy link
Copy Markdown
Member

@jangh-lee 감사합니다!! 깔끔합니다.. ^^ 이제 기여 과정에도 익숙해지신 것 같아요! 🥇

@seokho-son @jihoon-seo Good first issue 감사합니다. PR 진행했으나,
학습을 위해서 불필요한 브랜치 및 커밋이 많았어서 전부 삭제하고 올리는 것이라서 잘 되었는지 모르겠습니다.
검토 부탁드립니다.

커밋도 1개로 잘 정리해주셨고,
cloud-barista:main <- jangh-lee:main 이렇게 잘 해주셨어요.

(Upstream repository 입장에서는, jangh-lee:feature-xxx 이런식으로 하셔도 되고, jangh-lee:main 으로 PR을 주셔도 무관합니다.)

이제 다른 이슈들도 살펴보시고, 워크플로우, 소스 등도 살펴보시면서
재미있어 보이는 이슈들을 열거나 해결해보는 것도 좋을 것 같습니다.
살펴보시다가 어렵거나 까다로운 부분이 있으면 또 편히 알려주세요~~~ :)

@seokho-son

Copy link
Copy Markdown
Member

/lgtm

@github-actions github-actions Bot added the lgtm This PR is acceptable by at least one reviewer label Sep 28, 2021
@seokho-son

Copy link
Copy Markdown
Member

/approve

@seokho-son
seokho-son merged commit 930961f into cloud-barista:main Sep 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR is acceptable by at least one reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants