added ci for testing petl on github actions on all push and pull_request - Githubissues

petl-developers / petl

Python Extract Transform and Load Tables of Data

MIT License

1.24k stars 193 forks source link

added ci for testing petl on github actions on all push and pull_request #543

Closed juarezr closed 3 years ago

juarezr commented 3 years ago

This PR added ci for testing petl on github actions on all push and pull_request.

Changes

This performs the same tests already executing on travis and appveyor:

run nosetests testing for cases under test folder
run nosetest with doctest for embedded tests
run coverage tests with coveralls
run sphinx docs testing
run test on python versions 2.7 3.6 3.7 3.8 3.9
run on linux and windows

Also some improvements:

install dockers with postgresql and mysql for fromdb() todb() testing
install docker with sftp and samba for testing remote files read/write
run also on mac-os beyond ubuntu and windows
increased the test span on python 2.7 and 3.9
travis and appveyor still works

Caveats:

bcolz fails to build on CI and is skipped
doctest fails on some targes (works fully on python > 3.6 and ubuntu)
docker containers only available for ubuntu
no testing on arm architeture/targets
some warnings on xml test cases with recent lxml package
no support for release on pip/conda (currently on travis-ci)

Checklist

Checklist for for pull requests including new code and/or changes to existing code...

[ ] ~~Code~~
- [ ] ~~Includes unit tests~~
- [ ] ~~New functions have docstrings with examples that can be run with doctest~~
- [ ] ~~New functions are included in API docs~~
- [ ] ~~Docstrings include notes for any changes to API or behaviour~~
- [ ] ~~All changes documented in docs/changes.rst~~
[x] Testing
- [x] (Optional) Tested local against remote servers
- [x] Travis CI passes (unit tests run under Linux)
- [x] AppVeyor CI passes (unit tests run under Windows)
- [x] Unit test coverage has not decreased (see Coveralls)
[x] Changes
- [ ] ~~(Optional) Just a proof of concept~~
- [ ] (Optional) Work in progress
- [x] Ready to review
- [x] Ready to merge

coveralls commented 3 years ago

Coverage decreased (-0.6%) to 91.264% when pulling 53740c08cc69f4fa32198c3a11cca77d1d8b5c7b on juarezr:github_actions_ci into 02480fa35f6bf02a63eec217caddee25199952d3 on petl-developers:master.