-
-
Notifications
You must be signed in to change notification settings - Fork 554
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
Replace Travis with Github Actions #1158
Merged
Merged
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
f298808
add basic ci
ndanielsen 2dd02d5
move ot workflow folder
ndanielsen fadec9b
fix workflow
ndanielsen 8388dfc
fix workflow
ndanielsen f99e0ec
Update ci.yml
ndanielsen e37ee06
update
ndanielsen 8c52bd3
update
ndanielsen 7f1f88a
update
ndanielsen 4c35281
update
ndanielsen b975245
update
ndanielsen 0f891e6
update
ndanielsen f5645b7
update
ndanielsen d49f4ea
update
ndanielsen c558cb6
update
ndanielsen 861d222
update
ndanielsen d25a8bb
update
ndanielsen 7006a69
update
ndanielsen ee7e43d
update
ndanielsen ba6a50b
update
ndanielsen af14cbe
update
ndanielsen 6c442fd
update
ndanielsen 9771fab
update
ndanielsen 4dd0283
update
ndanielsen 9e731c6
update
ndanielsen a6fba30
update
ndanielsen 8f05d91
update
ndanielsen f09d25c
Fix conda (#11)
ndanielsen 3130f9b
add windows and slack notification (#12)
ndanielsen a44bda5
removed travis.yml and minor changes to ci.yml
a274685
Merge branch 'develop' of github.com:DistrictDataLabs/yellowbrick int…
53102ed
updated ci.yml
File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,97 @@ | ||
# This workflow will install Python dependencies, run tests and lint with a variety of Python versions | ||
# For more information see: https://help.github.com/actions/language-and-framework-guides/using-python-with-github-actions | ||
|
||
name: Python package | ||
|
||
on: [pull_request] | ||
|
||
jobs: | ||
test_conda: | ||
name: Conda (${{ matrix.python-version }}, ${{ matrix.os }}) | ||
runs-on: ${{ matrix.os }} | ||
strategy: | ||
fail-fast: false | ||
matrix: | ||
os: ["ubuntu-latest"] | ||
python-version: ["3.7", "3.8"] | ||
steps: | ||
- uses: actions/checkout@v2 | ||
- uses: conda-incubator/setup-miniconda@v2 | ||
with: | ||
auto-update-conda: true | ||
python-version: ${{ matrix.python-version }} | ||
channels: conda-forge,spyder-ide | ||
channel-priority: flexible | ||
|
||
- name: Conda info | ||
shell: bash -l {0} | ||
run: conda info | ||
- name: Add conda to system path | ||
shell: bash -l {0} | ||
run: | | ||
# $CONDA is an environment variable pointing to the root of the miniconda directory | ||
echo $CONDA/bin >> $GITHUB_PATH | ||
- name: Install dependencies | ||
shell: bash -l {0} | ||
run: | | ||
conda config --set always_yes yes --set changeps1 no | ||
conda update -n base conda --yes | ||
conda env create -f tests/requirements.txt -n yellowbrick python=${{ matrix.python-version }} | ||
- name: Test with pytest | ||
shell: bash -l {0} | ||
run: | | ||
conda activate yellowbrick | ||
python -m nltk.downloader popular | ||
make test | ||
|
||
- name: Upload coverage to Codecov | ||
uses: codecov/codecov-action@v1 | ||
with: | ||
env_vars: OS,PYTHON | ||
name: codecov-umbrella | ||
fail_ci_if_error: false | ||
|
||
|
||
test_pypi: | ||
name: PyPi (${{ matrix.python-version }}, ${{ matrix.os }}) | ||
strategy: | ||
fail-fast: false | ||
matrix: | ||
python-version: [3.7, 3.8] | ||
os: [ubuntu-latest, macos-latest, windows-latest] | ||
|
||
runs-on: ${{ matrix.os }} | ||
|
||
steps: | ||
- uses: actions/checkout@v2 | ||
- name: Set up Python ${{ matrix.python-version }} | ||
uses: actions/setup-python@v2 | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
- name: Install dependencies | ||
run: | | ||
python -m pip install --upgrade pip | ||
pip install -r tests/requirements.txt | ||
python -m pip install coveralls | ||
- name: Test with pytest | ||
run: | | ||
python -m nltk.downloader popular | ||
make test | ||
- name: Upload coverage to Codecov | ||
uses: codecov/codecov-action@v1 | ||
with: | ||
env_vars: OS,PYTHON | ||
name: codecov-umbrella | ||
fail_ci_if_error: false | ||
|
||
|
||
slackNotification: | ||
name: Slack Notification | ||
needs: [test_pypi, test_conda] | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v2 | ||
- name: Slack Notification | ||
uses: rtCamp/action-slack-notify@v2 | ||
env: | ||
SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK }} |
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,7 +14,7 @@ cycler==0.10.0 | |
pytest==6.1.1 | ||
pytest-cov==2.10.1 | ||
pytest-flakes==4.0.2 | ||
pytest-spec==3.0.4 | ||
pytest-spec>=2.0.0 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. downgrade to allow conda to work There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. thanks for making this fix! |
||
coverage==5.3 | ||
|
||
# Optional Testing Dependencies | ||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
allow codecov integration to work