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

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

Merged
merged 1 commit into from
Sep 28, 2021

Conversation

jangh-lee
Copy link
Contributor

#797 이슈입니다.

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

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

@seokho-son
Copy link
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
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
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