Skip to content

Commit c962b59

Browse files
authored
Remove test duplication in CI (#151)
* One test run per job * second attempt * Update step name * Update changelog.rst
1 parent a1a71e4 commit c962b59

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

.github/workflows/unit_tests_with_latest_deps.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,15 +37,15 @@ jobs:
3737
make installdeps-test
3838
pip install git+https://github.com/alteryx/featuretools.git@main#egg=featuretools --force-reinstall
3939
- if: ${{ matrix.featuretools_version == 'Release' }}
40-
name: Install latest version of Featuretools from main branch
40+
name: Install latest version of Featuretools from PyPI
4141
run: |
4242
make installdeps-complete
4343
make installdeps-test
4444
- if: ${{ matrix.python_version != 3.7 || matrix.featuretools_version != 'main' }}
4545
name: Run unit tests with no code coverage
4646
run: |
4747
make test
48-
- if: ${{ matrix.python_version == 3.7 || matrix.featuretools_version == 'main' }}
48+
- if: ${{ matrix.python_version == 3.7 && matrix.featuretools_version == 'main' }}
4949
name: Run unit tests with code coverage
5050
run: |
5151
make testcoverage

docs/source/changelog.rst

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ Future Release
1212
* Documentation Changes
1313
* Testing Changes
1414
* Fix latest dependency checker to create PR (:pr:`129`)
15+
* Fixed unit tests workflow test choice logic (:pr:`151`)
1516

1617
Thanks to the following people for contributing to this release:
1718
:user:`gsheni`, :user:`rwedge`

0 commit comments

Comments
 (0)