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.

Sign up for GitHub

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

Jump to bottom

CI Updates and Improvements #853

Closed
9 tasks done
devin-petersohn opened this issue Nov 2, 2019 · 1 comment
Closed
9 tasks done

CI Updates and Improvements #853

devin-petersohn opened this issue Nov 2, 2019 · 1 comment
Labels
Testing 📈 Issues related to testing
Milestone
0.7.0

Comments

@devin-petersohn
Copy link
Collaborator

devin-petersohn commented Nov 2, 2019 •
edited

This is a meta-issue for the CI. The purpose of this issue is to track all of the changes we would like to make to the CI to improve developer experience and turnaround. Some of these tasks are also related to the quality of tests for correctness.

  • Add Dask to testing environment
  • Add Codecov Token
  • Add Windows to the CI test matrix
  • [ ] Add Mac OSX to the CI test matrix
  • Improve test granularity (Bin tests into relevant groups rather than arbitrary groups)
  • Add caching to GitHub Actions
  • Deprecate Travis
  • Update Pytest
  • Only test functionality that was updated
  • Remove unused tests
  • [ ] Test Python 3.8
@devin-petersohn devin-petersohn added the Testing 📈 Issues related to testing label Nov 2, 2019
@devin-petersohn devin-petersohn mentioned this issue Nov 2, 2019
Merged
@devin-petersohn devin-petersohn added this to the 0.6.4 milestone Dec 10, 2019
devin-petersohn added a commit to devin-petersohn/modin that referenced this issue Dec 12, 2019
@devin-petersohn
Update pytest and clean up tests a bit
62c83f9 
* Contributes to modin-project#853
@devin-petersohn devin-petersohn mentioned this issue Dec 12, 2019
Merged
3 tasks
devin-petersohn added a commit that referenced this issue Dec 13, 2019
@devin-petersohn
Update pytest and clean up tests a bit (#903)
1c40056 
* Update pytest and clean up tests a bit

* Contributes to #853

* Lint

* Fix outdated use of pytest.raises
devin-petersohn added a commit to devin-petersohn/modin that referenced this issue Dec 31, 2019
@devin-petersohn
Begin testing only files that have changed
5e431d0 
* Progress on modin-project#853
* Uses testmon, which looks at the coverage and runs tests that cover
  lines that have changed.
* Restructure and clean up .yml files
* Split push and PR into separate workflows
@devin-petersohn devin-petersohn mentioned this issue Dec 31, 2019
Merged
2 tasks
devin-petersohn added a commit that referenced this issue Dec 31, 2019
@devin-petersohn
Begin testing only files that have changed (#924)
10585f0 
* Begin testing only files that have changed

* Progress on #853
* Uses testmon, which looks at the coverage and runs tests that cover
  lines that have changed.
* Restructure and clean up .yml files
* Split push and PR into separate workflows

* Wait to test download

* Remove xdist args

* Remove codecov from PRs

* Fix
@devin-petersohn devin-petersohn mentioned this issue Jan 3, 2020
Merged
3 tasks
devin-petersohn added a commit to devin-petersohn/modin that referenced this issue Jan 4, 2020
@devin-petersohn
Set pure=False for Dask Client.submit
7914df2 
* Resolves modin-project#956
* Add Dask tests to CI and master test suite, progress toward modin-project#853
* Update run-tests.sh to run all Dask tests

Lint

Lint

Update scatter as well
devin-petersohn added a commit that referenced this issue Jan 4, 2020
@devin-petersohn
Set pure=False for Dask Client.submit and hash=False for `… (#957)
28801f0 
* Set `pure=False` for Dask `Client.submit`

* Resolves #956
* Add Dask tests to CI and master test suite, progress toward #853
* Update run-tests.sh to run all Dask tests
* Progress toward #853.
* Add skip code for Windows, related to issue #960
devin-petersohn added a commit to devin-petersohn/modin that referenced this issue Jan 4, 2020
@devin-petersohn
Make test_dataframe.py more granular with test suite
d19c7a4 
* Progress toward modin-project#853
* Adds more tests to the matrix to take better advantage of concurrent
  builds
* Classify tests into categories better
@devin-petersohn devin-petersohn mentioned this issue Jan 4, 2020
Merged
3 tasks
devin-petersohn added a commit that referenced this issue Jan 5, 2020
@devin-petersohn
Make test_dataframe.py more granular with test suite (#966)
11d7bd8 
* Make test_dataframe.py more granular with test suite

* Progress toward #853
* Adds more tests to the matrix to take better advantage of concurrent
  builds
* Classify tests into categories better
* Update granularity to make tests runtime ~10min
devin-petersohn added a commit to devin-petersohn/modin that referenced this issue Jan 5, 2020
@devin-petersohn
Cache pip depdendencies between builds to speed up process
c3b7308 
* Progress toward modin-project#853
* Adds caching to each build
@devin-petersohn devin-petersohn mentioned this issue Jan 5, 2020
Merged
3 tasks
devin-petersohn added a commit that referenced this issue Jan 5, 2020
@devin-petersohn
Cache pip depdendencies between builds to speed up process (#967)
f0b6b39 
* Cache pip depdendencies between builds to speed up process

* Progress toward #853
* Adds caching to each build
@devin-petersohn devin-petersohn mentioned this issue Jan 5, 2020
Merged
3 tasks
@devin-petersohn
Copy link
Collaborator Author

Moving Python 3.8 testing to a new issue. Closing as resolved via #850, #903, #924, #957, #966, #967, #968

@devin-petersohn devin-petersohn mentioned this issue Jan 6, 2020
Closed
@smurching smurching mentioned this issue Jan 29, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Testing 📈 Issues related to testing
Projects
None yet
Milestone
0.7.0
Development

No branches or pull requests
1 participant
@devin-petersohn