Skip to content

Commit 11837a5

Browse files
authored
Change codecov v3 for GitHub workflow (#184)
* Update unit_tests_with_latest_deps.yml * Update changelog.rst * Update setup.cfg * Update unit_tests_with_latest_deps.yml * Update unit_tests_with_latest_deps.yml * Update unit_tests_with_latest_deps.yml * Update setup.cfg * Update unit_tests_with_latest_deps.yml * Update README.md * Update README.md
1 parent 5fd1d45 commit 11837a5

File tree

3 files changed

+13
-7
lines changed

3 files changed

+13
-7
lines changed

.github/workflows/unit_tests_with_latest_deps.yml

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ jobs:
1616
matrix:
1717
os: [ubuntu-latest, macos-latest]
1818
python_version: ["3.8", "3.9", "3.10"]
19-
featuretools_version: ["Release", "Main"]
19+
featuretools_version: ["release", "main"]
2020
steps:
2121
- name: Set up python ${{ matrix.python_version }}
2222
uses: actions/setup-python@v2
@@ -30,13 +30,13 @@ jobs:
3030
- name: Install requirements
3131
run: |
3232
pip config --site set global.progress_bar off
33-
- if: ${{ matrix.featuretools_version == 'Main' }}
33+
- if: ${{ matrix.featuretools_version == 'main' }}
3434
name: Install latest version of Featuretools from main branch
3535
run: |
3636
make installdeps-complete
3737
make installdeps-test
3838
pip install git+https://github.com/alteryx/featuretools.git@main#egg=featuretools --force-reinstall
39-
- if: ${{ matrix.featuretools_version == 'Release' }}
39+
- if: ${{ matrix.featuretools_version == 'release' }}
4040
name: Install latest version of Featuretools from PyPI
4141
run: |
4242
make installdeps-complete
@@ -48,10 +48,11 @@ jobs:
4848
- if: ${{ matrix.python_version == 3.8 && matrix.featuretools_version == 'main' }}
4949
name: Run unit tests with code coverage
5050
run: |
51-
make testcoverage
51+
pytest nlp_primitives/ --cov=nlp_primitives/ --cov-config=pyproject.toml --cov-report=xml:coverage.xml
5252
- if: ${{ matrix.python_version == 3.8 && matrix.featuretools_version == 'main' }}
5353
name: Upload coverage to Codecov
54-
uses: codecov/codecov-action@v1
54+
uses: codecov/codecov-action@v3
5555
with:
5656
fail_ci_if_error: true
57-
token: ${{ secrets.CODECOV_TOKEN }}
57+
files: ${{ github.workspace }}/coverage.xml
58+
verbose: true

README.md

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
# NLP Primitives
22

33
<p align="center">
4+
<a href="https://codecov.io/gh/alteryx/nlp_primitives">
5+
<img src="https://codecov.io/gh/alteryx/nlp_primitives/branch/main/graph/badge.svg"/>
6+
</a>
47
<a href="https://github.com/alteryx/nlp_primitives/actions?query=branch%3Amain+workflow%3ATests" target="_blank">
58
<img src="https://github.com/alteryx/nlp_primitives/workflows/Tests/badge.svg?branch=main" alt="Tests" />
69
</a>
@@ -23,7 +26,8 @@ nlp_primitives is a Python library with Natural Language Processing Primitives,
2326

2427
nlp_primitives allows you to make use of text data in your machine learning pipeline in the same pipeline as the rest of your data.
2528

26-
### Install
29+
## Installation
30+
2731
There are two options for installing nlp_primitives. Both of the options will also install Featuretools if it is not already installed.
2832

2933
The first option is to install a version of nlp_primitives that does not include Tensorflow. With this option, primitives that depend on Tensorflow cannot be used. Currently, the only primitive that can not be used with this install option is ``UniversalSentenceEncoder``.

docs/source/changelog.rst

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ Future Release
1414
* Change `TitleWordCount`, `PunctuationCount`, `UpperCaseCount` to use `CountString` (:pr:`183`)
1515
* Documentation Changes
1616
* Testing Changes
17+
* Change codecov v3 for GitHub workflow (:pr:`184`)
1718

1819
Thanks to the following people for contributing to this release:
1920
:user:`gsheni`, :user:`sbadithe`

0 commit comments

Comments
 (0)