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

refactor: CI 통합 #42

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

refactor: CI 통합 #42

wants to merge 5 commits into from

Conversation

Lapis0875
Copy link
Contributor

style, type -> test -> deploy 순으로 진행되도록 CI를 한개 파일로 합쳤습니다.

@Lapis0875 Lapis0875 added the enhancement New feature or request label Aug 13, 2022
@Lapis0875 Lapis0875 requested a review from a team as a code owner August 13, 2022 12:26
Comment on lines +76 to +96
deploy:
runs-on: ubuntu-latest
needs: [ style, type, build ]
steps:
- uses: actions/checkout@v2
- name: Set up Python
uses: actions/setup-python@v2
with:
python-version: '3.x'
- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install -r requirements.txt
pip install setuptools wheel twine
- name: Build and publish
env:
TWINE_USERNAME: ${{ secrets.PYPI_USERNAME }}
TWINE_PASSWORD: ${{ secrets.PYPI_PASSWORD }}
run: |
python setup.py sdist bdist_wheel
twine upload dist/*
Copy link
Contributor

Choose a reason for hiding this comment

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

이 프로세스는 빌드가 생성 될시에만 실행이 되나요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

우선 build에 의존시켜뒀으니, 그럴거라고 생각합니다. 다만 테스트에서 오류가 나 이후의 동작은 확인하지 못했습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

테스트 오류 해결 후 확인했습니다. build까지 동작 이후에 deploy가 동작합니다.

Copy link
Contributor

@SaidBySolo SaidBySolo Sep 30, 2022

Choose a reason for hiding this comment

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

배포는 릴리즈 이벤트가 발생될때만 작동하게 하면 될것같아요 :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아아 그러네요!

@codecov
Copy link

codecov bot commented Sep 30, 2022

Codecov Report

Merging #42 (35dc138) into master (5d98002) will decrease coverage by 1.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
- Coverage   66.66%   65.33%   -1.34%     
==========================================
  Files          10        9       -1     
  Lines         390      375      -15     
==========================================
- Hits          260      245      -15     
  Misses        130      130              
Impacted Files Coverage Δ
koreanbots/__init__.py

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants